Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Display event alarms in calendar #51

Closed
georgehrke opened this issue Sep 18, 2016 · 9 comments
Closed

Display event alarms in calendar #51

georgehrke opened this issue Sep 18, 2016 · 9 comments
Labels
1. to develop Accepted and waiting to be taken care of enhancement New feature request skill:frontend Issues and PRs that require JavaScript/Vue/styling development skills
Milestone

Comments

@georgehrke
Copy link
Member

georgehrke commented Sep 18, 2016

https://tools.ietf.org/html/rfc4791#section-7.8.5 might come handy here

https://developer.mozilla.org/en-US/docs/Web/API/notification


Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

@georgehrke georgehrke added 1. to develop Accepted and waiting to be taken care of enhancement New feature request skill:frontend Issues and PRs that require JavaScript/Vue/styling development skills labels Sep 18, 2016
@georgehrke georgehrke added this to the 1.7 milestone Sep 18, 2016
@cwmke
Copy link

cwmke commented Jul 31, 2018

The current discussion seems to be going towards notifications showing up through the Nextcloud Notifications app. Would this still send notifications through the browser so that they show up in my OS's notification area?

@georgehrke
Copy link
Member Author

@cwmke I don't think native OS notifications should be implemented solely in the calendar app. They should be available for any notification.

@nickvergessen Are there plans to use the Browser Notification API with the notifications app? :)

@cwmke
Copy link

cwmke commented Jul 31, 2018

@georgehrke I agree. That would be great to have all Nextcloud notifications integrated with the browser's api.

@nickvergessen
Copy link
Member

Are there plans to use the Browser Notification API with the notifications app? :)

This is done since 2015, sorry you are late to the party…

@nickvergessen
Copy link
Member

Please keep in mind, that when you publish notifications, this will also trigger push notifications on our apps and the desktop client.
So this should be optional at least, because otherwise you receive duplicate notifications on each device (your calendar app/client and your nextcloud app/client) (or even three times when you have the website also open)

@cwmke
Copy link

cwmke commented Jul 31, 2018

@nickvergessen When you dismiss a notification on one client is it dismissed in the others as well?

@nickvergessen
Copy link
Member

Well "yes", but the sync client might have already triggered the desktop notification and the your calendar and nextcloud app on the phone also create the notifications at the same time.

@cwmke
Copy link

cwmke commented Aug 1, 2018

@georgehrke: To me it seems to make more sense to treat Calendar as any other calendar client that has it's own notifications separate from Nextcloud Notifications similarly to how Evolution or Thunderbird functions.

@georgehrke
Copy link
Member Author

This was implemented in the server and will be part of Nextcloud 17.
nextcloud/server#3044

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop Accepted and waiting to be taken care of enhancement New feature request skill:frontend Issues and PRs that require JavaScript/Vue/styling development skills
Projects
None yet
Development

No branches or pull requests

3 participants