-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Fix carddav activities #33617
Fix carddav activities #33617
Conversation
Why does it work for activities of type |
Because they have different settings https://github.com/nextcloud/server/tree/814d9ac74a7d673746dc2a2760a2b8f773523a08/apps/dav/lib/CalDAV/Activity/Setting But it was requested to not do that for contacts. The main of caldav being different is also to be able to ignore todo changes while being notified about events. |
/backport to stable24 |
/backport to stable23 |
|
The settings where combined last minute but at the same time the activities where not adjusted to map an existing setting so the filter was not possible to even limit it to the types that the activities had. Signed-off-by: Joas Schilling <[email protected]>
786fef6
to
eab5189
Compare
The backport to stable24 failed. Please do this backport manually. |
The backport to stable23 failed. Please do this backport manually. |
The settings where combined last minute but at the same time the activities
where not adjusted to map an existing setting so the filter was not possible
to even limit it to the types that the activities had.
Unluckily this is not retro-active
Fix nextcloud/activity#856