server icon indicating copy to clipboard operation
server copied to clipboard

fix(dav): fix event birthday alarms not being updated

Open tcitworld opened this issue 1 year ago • 7 comments

In https://github.com/nextcloud/server/pull/33251 the default offset for a birthday event alarm was changed to 9AM on the day of the event.

However the birthdayEvenChanged method did not account for alarm changes, so it was never propagated to existing birthday events, which were kept on midnight.

Checklist

tcitworld avatar Aug 19 '23 09:08 tcitworld

/backport to stable27

tcitworld avatar Aug 19 '23 09:08 tcitworld

/backport to stable26

tcitworld avatar Aug 19 '23 09:08 tcitworld

/backport to stable25

tcitworld avatar Aug 19 '23 09:08 tcitworld

Should we add a repair job just like RegenerateBirthdayCalendars to fix all existing events? Or just edit the app config value from regeneratedBirthdayCalendarsForYearFix to something like regeneratedBirthdayCalendarsForAlarmFix so that it's triggered again?

tcitworld avatar Oct 02 '23 09:10 tcitworld

Should we add a repair job just like RegenerateBirthdayCalendars to fix all existing events? Or just edit the app config value from regeneratedBirthdayCalendarsForYearFix to something like regeneratedBirthdayCalendarsForAlarmFix so that it's triggered again?

Repair step sounds good to me.

miaulalala avatar Oct 02 '23 12:10 miaulalala

Rebased

tcitworld avatar Feb 01 '24 17:02 tcitworld

/backport to stable28

tcitworld avatar Feb 01 '24 17:02 tcitworld

psalm says No Way Snl GIF

skjnldsv avatar Feb 23 '24 15:02 skjnldsv