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

dav: fix birthday calendar event generation for leap days #29010

Merged

Conversation

mattian
Copy link
Contributor

@mattian mattian commented Sep 30, 2021

Fixes #19218.
Fix generation of birthday calendar events for the 29th of February.

Copy link
Contributor

@miaulalala miaulalala left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tcitworld
Copy link
Member

Please rebase

@mattian mattian force-pushed the fix/19218/birthday-leap-day branch from 605e430 to ac0627b Compare July 28, 2022 20:47
@szaimen szaimen added this to the Nextcloud 25 milestone Jul 28, 2022
This was referenced Aug 12, 2022
This was referenced Aug 24, 2022
This was referenced Sep 6, 2022
@skjnldsv skjnldsv mentioned this pull request Sep 15, 2022
This was referenced Sep 20, 2022
@blizzz blizzz modified the milestones: Nextcloud 25, Nextcloud 26 Sep 22, 2022
Copy link
Member

@ChristophWurst ChristophWurst left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mattian please rebase

@miaulalala
Copy link
Contributor

miaulalala commented Oct 20, 2022

Drone failing with:

--- Failed scenarios:
--
512 |  
513 | /drone/src/tests/acceptance/features/app-files-sharing.feature:353

unrelated.

Performance testing is only checking for origin remote.

@ChristophWurst ChristophWurst added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Jan 12, 2023
@blizzz blizzz mentioned this pull request Feb 1, 2023
Fix generation of birthday calendar events for the 29th of February.

A recurring event for the 29th of February in the (default) Gregorian
calendar system would only generate instances in leap years. Fix this
behaviour by generating recurring events for the last day of February,
the 29th on leap years and the 28th otherwise.

Signed-off-by: Mattia Narducci <mattianarducci1@gmail.com>
@miaulalala
Copy link
Contributor

@blizzz drone failure unrelated, force merge would be apprechiated.

@miaulalala
Copy link
Contributor

thanks @mattian 😄

@ChristophWurst ChristophWurst merged commit ee05e01 into nextcloud:master Feb 8, 2023
@blizzz blizzz mentioned this pull request Feb 9, 2023
@mattian mattian deleted the fix/19218/birthday-leap-day branch April 5, 2023 20:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish enhancement feature: dav
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Contact's birthday on leap day shows up on March 1 instead of February 29
6 participants