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

Update holidays Nederland #215

Closed
7 tasks done
PanderMusubi opened this issue Apr 22, 2024 · 4 comments
Closed
7 tasks done

Update holidays Nederland #215

PanderMusubi opened this issue Apr 22, 2024 · 4 comments
Labels
enhancement New feature or request

Comments

@PanderMusubi
Copy link

Checklist

  • I can reproduce the bug with the latest version given here.
  • I made sure that there are no existing issues - open or closed - to which I could contribute my information.
  • I made sure that there are no existing discussions - open or closed - to which I could contribute my information.
  • I have read the FAQs inside the app (Menu -> About -> FAQs) and my problem isn't listed.
  • I have taken the time to fill in all the required details. I understand that the bug report will be dismissed otherwise.
  • This issue contains only one bug.
  • I have read and understood the contribution guidelines.

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

  1. Go to menu
  2. Choose add holidays
  3. Choose Nederland (Netherlands)

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.

@PanderMusubi PanderMusubi added bug Something is not working needs triage Issue is not yet ready for PR authors to take up labels Apr 22, 2024
@Aga-C
Copy link
Member

Aga-C commented Apr 22, 2024

Holidays will be updated automatically since the next version.

@Aga-C Aga-C closed this as completed Apr 22, 2024
@Aga-C Aga-C added enhancement New feature or request and removed bug Something is not working needs triage Issue is not yet ready for PR authors to take up labels Apr 22, 2024
@PanderMusubi
Copy link
Author

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.

@tswistak
Copy link
Contributor

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)

@PanderMusubi
Copy link
Author

Thank you for the explanation. That is another source and I will contact them to make some improvements and additions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants