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

Liberation Day is official in the Netherlands #169

Merged
merged 1 commit into from
Sep 4, 2019

Conversation

c960657
Copy link
Contributor

@c960657 c960657 commented Sep 3, 2019

Liberation Day in the Netherlands is a public holiday, according to Wikipedia and Dutch law (not mentioned by named but referred to as fifth of May).

@stelgenhof
Copy link
Member

stelgenhof commented Sep 4, 2019

As the notes on Wikipedia said, May the 5th is not mandatory paid off for everybody. That's perhaps why initially I marked it as 'OBSERVANCE'. But now thinking about it may be better to mark it as OFFICIAL indeed.

@stelgenhof stelgenhof added this to the v2.2.0 milestone Sep 4, 2019
@stelgenhof stelgenhof merged commit 3b036af into azuyalabs:develop Sep 4, 2019
@c960657 c960657 deleted the liberation-day-nl branch September 4, 2019 15:13
@c960657
Copy link
Contributor Author

c960657 commented Sep 4, 2019

The distinction between the different holiday types is difficult due to different traditions in each country.

Many countries have legislation containing an explicit list of holidays. In my understanding, these have TYPE_OFFICIAL. Whether these days are paid/non-paid days off is a different matter – this often depends on tradition, collective agreements. With respect to non-working days, legislation often covers only the public sector.

Often the legislation mentions other days that are not official holidays but in some contexts have the same status. E.g. article 3.1 in the Dutch law mentioned above mentions Good Friday as such a day. These are probably TYPE_OBSERVANCE.

I assume the official classification is not so relevant for users of Yasumi. Instead I assume they are more interested in – depending on the specific use-case – whether it is a regular working day, whether shops are open, whether banks are open, etc. I am not sure how to model this, but I have outlined some thoughts in #99.

@stelgenhof
Copy link
Member

Yes correct. It may not always be easy to assign the proper indication. Furthermore the use-cases might be different for every user of Yasumi, so I tried to keep it as straightforward as possible (and think it should be that). For the moment this simple classification works but also understand and agree the proposal you made in #99 . I haven't yet properly thought about :(

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

Successfully merging this pull request may close these issues.

None yet

2 participants