azuyalabs / yasumi

The easy PHP Library for calculating holidays
https://www.yasumi.dev
Other
1.04k stars 152 forks source link

Spelling fixes for da_DK #96

Closed c960657 closed 6 years ago

c960657 commented 6 years ago

Fix a few language issues for da_DK:

The provider currently supports what are usually known as holidays (hellidage) in Denmark. Other days are also important: 1) Lukkeloven (the Opening Hours Act) is the law defines when shops are allowed to stay open. In addition to the above mentioned holidays, the law prohibits most shops from staying open on Constitution Day (5 June), Christmas Eve, New Year's Eve (partially) after 15:00, so shop staff have these days off. Others usually do not work on Christmas Eve and New Year's Eve, though this is not covered by any legislation. 2) Labour Day (1 May) is not mentioned in any legislation, but for people working under a trade union agreement (80% of the work force does that), either Constitution Day or Labour Day (depending on local/industry agreements) is a day off (except for police officers, nurses etc. who also work on weekends).

Would these days be appropriate to include in Yasumi? Which type should they classified as?

stelgenhof commented 6 years ago

Hi @c960657

Thank you very much for your PR and the extensive information. Really appreciate that! As for your question regarding 'other' holidays, in Yasumi you can currently classify days in 5 different ways:

This classification covers I think most of the situations however there are always exceptions :). In your case I would classify these days as 'observance'.

Note: The Yasumi classification is merely a category and has no particular logic associated. The classification can be helpful in your project where you may have some business logic related to certain holiday types.

Hopefully that anwwers your question.

Cheers! Sacha

c960657 commented 6 years ago

Thanks for clarifying. I'll make a separate PR for adding those days.

stelgenhof commented 6 years ago

@c960657 Sorry for the delay. I thought you were going to make additional changes in this PR. I misread you are going to create additional PR's :) I'll review it once more and merge it ASAP.

c960657 commented 6 years ago

I have filed a separate PR for the observed holidays: #104