-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Enable caldav push notifications by default #39862
Comments
In a way this is "breaking" because systems will behave different after this has been deployed. Do we want this change to be included only in 28 or 27.1 as well @AndyScherzinger @jancborchardt? |
I’m fine with either, but I would say 27.1 to get it done? |
27.1 would also be fine with me 👍 |
#40103 should be ready for merge now. We are at RC3 of 27.1. I would veto merging this for 27 still because we wouldn't have time to roll back if there are issues with this change. |
I hope this can be disables per settings in the feature. I don't have access to occ and therefore can not disable it ...! |
How to use GitHub
Is your feature request related to a problem? Please describe.
As a user of Nextcloud who doesn't use connected clients I want to see a web notification for calendar events.
Describe the solution you'd like
Revert #16970
Describe alternatives you've considered
#39158
Additional context
Needs admin upgrade docs because it will have a big influence on existing setups.
The text was updated successfully, but these errors were encountered: