-
-
Notifications
You must be signed in to change notification settings - Fork 22
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
Support additional holiday types - bank holidays and observance #537
Comments
@kmcbride3 thanks for your input. I really appreciate it! I would suggest the following Public: Public holiday
We will declare all of these as Public Holiday Bank: Bank holiday, banks and offices are closed Observance Observance, is a celebration or commemoration that doesn't include a day off from work. There are different types of observance like Religious, Secular, Awareness, International or National observance. We will declare all of these as Observance Unofficial: Holidays which are not public or bank holidays (Deprecated) Maybe this types are enough to start and we can add more if we need them. Or are there must haves? |
@derTobsch I like it. I can see many people considering 'School' as a must-have, for example if they're using Jollyday in their home automation system and want to build rules around whether or not it's a 'school' day. On the other hand, those dates would vary from location to location, even down to city level (and possibly lower - school district or even per school themselves)... plus they could change from year-to-year. As a compromise, perhaps the holiday.xsd definition should support 'School' as a type, but given the specificity of this type, we don't define those holiday types in each country's holiday definitions -- end users would have to augment them on their own with the school dates relevant to their use case. For clarity on Public Holidays, would that definition mean that some previously unofficial holidays now become public holidays if they meet the definition as "days when most of the public enjoys a non-working days" if local laws designate them as non-working days, though they do not require employees to be paid for them? Thinking specifically about "prescribed days of rest" in New Brunswick (NB), Canada, where provincial law declares them as non-working days, however they are not statutory (that is, employers are not obligated to pay employees for these non-working days, although many do). If the intent is to keep "Public Holidays" strictly to where most of the public enjoys a paid non-working day as determined by local laws, then I would recommend including 'paid' in the Public Holiday definition - such as:
If the originally-proposed definition for Public Holidays is correct (as in, it doesn't matter whether people are paid for these non-working days or not), I'm good with that too. I suggest this definition for Bank holidays:
I also wonder if including UK's bank holidays as Public holidays, and then having a separate holiday type of 'Bank' would be confusing for some. Would something akin to 'Government' holiday be more apt? |
Hey @kmcbride3, I would like to start without school holidays for the moment. We can add them later if we need them. I thought the same about the naming of 'bank holidays' and I am not sure if there is a better "official" naming at the moment. I need to take a deeper look at that. |
Works for me, thank you for considering my feedback! I fully agree about the naming of 'bank holidays' - I couldn't think of a better name myself that would be more recognized/official. |
More information
Building on #482 and #462, which redefine 'official' holidays as 'public' holidays and introduce a new 'bank' holiday type respectively, a discussion should be had on what, if any, other holiday types should be considered for inclusion.
Assuming PR #467 goes through, Jollyday would support the following holiday types:
The original PR suggests types "Schools" (where schools are closed) and "Authorities" (where government offices/services are closed).
For consideration:
The text was updated successfully, but these errors were encountered: