-
Notifications
You must be signed in to change notification settings - Fork 64
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
Update holidays Nederland #215
Comments
Holidays will be updated automatically since the next version. |
Nice! What do you use as upstream? For the Netherlands, please don't use the one offered by Thunderbird. They use the same upstream eventually, but can also lag behind as they don't check for updates automatically. Additionally, Thunderbird has changed the way they generate holiday ICS files and caused also for missing holidays for the Netherlands. This is an ongoing issue, so sidestep them for NL for the holiday calendar. |
I will answer, since I implemented it. We don't use Thunderbird's database, but https://github.com/commenthol/date-holidays. Here's explanation why: #152 (comment) |
Thank you for the explanation. That is another source and I will contact them to make some improvements and additions. |
Checklist
Affected app version
1.0.3
Affected Android/Custom ROM version
Android 11 / LineageOS 18.1
Affected device model
OnePlus 8T
How did you install the app?
F-Droid / IzzyOnDroid
Steps to reproduce the bug
Expected behavior
Up to date Dutch holidays
Actual behavior
Out of date Dutch holidays. It contains Queensway 30 April 2024, while the last was in 2013. Also some new additions are missing.
The app source file
https://github.com/FossifyOrg/Calendar/blob/master/app/src/main/assets/netherlands.ics
should be a copy of the upstream version
https://github.com/PanderMusubi/dutch-holidays/blob/master/NederlandseFeestdagen.ics
Please add in the build process an automated check if this file needs to be updated with upstream.
Screenshots/Screen recordings
No response
Additional information
Please give this issue a high priority as outdated holiday info can have costly consequences.
The text was updated successfully, but these errors were encountered: