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

Incorrect German holidays #74

Closed
DijanaPenic opened this issue Sep 21, 2022 · 2 comments
Closed

Incorrect German holidays #74

DijanaPenic opened this issue Sep 21, 2022 · 2 comments

Comments

@DijanaPenic
Copy link

I've checked holidays for year 2022 and here are my findings:

June 5th (Pentecost)

  • not returned

November 16th (Repentance and prayer day)

  • library returns November 23rd

September 20th (World's children day)

  • library hardcodes it to September 20th, 2019
if (this.HasWorldChildrensDay(2019))
    dateTimeList.Add(GermanPublicHoliday.WorldChildrensDay(2019));

July 17th (Easter Sunday)

  • not returned

Package version: v2.16.2
Tested method: IList PublicHolidays(int year)

martinjw added a commit that referenced this issue Sep 21, 2022
@martinjw
Copy link
Owner

Pentecost + Easter Sundays are Sundays, so not normally working days. Checking on this, they are not public holidays in most german Lander - only Brandenburg legally. As a Sunday, it is not useful to include it.

Repentance day had a bad algorithm for first advent. Should be fixed now

Children's Day is fixed too.

Thanks!

@martinjw
Copy link
Owner

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

No branches or pull requests

2 participants