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

Feature Request: Additionally support public holidays to determine school holidays #4

Closed
Marty56 opened this issue Nov 1, 2019 · 5 comments

Comments

@Marty56
Copy link

Marty56 commented Nov 1, 2019

It would be great if public holidays would be treated like school holidays.

Just remote the filtering of public holidays by altering one line of code in main
//const arrOnlyholiday = arrNewHoliday.filter(d => d.starts_on != d.ends_on);
const arrOnlyholiday = arrNewHoliday;

@simatec
Copy link
Owner

simatec commented Mar 23, 2020

added at release 0.5.0

@simatec simatec closed this as completed Mar 23, 2020
@Marty56
Copy link
Author

Marty56 commented Mar 23, 2020 via email

@simatec
Copy link
Owner

simatec commented Mar 23, 2020

Daten komme auch alle wieder?

@Marty56
Copy link
Author

Marty56 commented Mar 23, 2020

Ich habe es nur für Bayern getestet. Das wurden aktuelle Coronaschließung und als nächste Ferien Osterferien angezeigt. Das war ok.

@simatec
Copy link
Owner

simatec commented Mar 23, 2020

So ist es bei mir auch aktuell. Sachsen gibt dann aber noch den Karfreitag aus, da die Osterferien dort eine Woche später sind. Voraussetzung ist allerdings, dass die Feiertage nicht ignoriert werden. Im nächsten Schritt will ich mal schauen, dass wir noch mehr Auswahl und nicht nur die Bundesländer bekommen

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