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

Spelling fixes for da_DK #96

Merged
merged 2 commits into from
Apr 23, 2018
Merged

Spelling fixes for da_DK #96

merged 2 commits into from
Apr 23, 2018

Conversation

c960657
Copy link
Contributor

@c960657 c960657 commented Mar 4, 2018

Fix a few language issues for da_DK:

  • Holiday names are not proper nouns and they should thus not be written in Title Case (except Kristi himmelfartsdag because Kristi (Christ) is a proper noun). This is according to the official Danish ortography (source: § 12.13.c in Retskrivningsordbogen – the authoritative reference for the Danish language), though Title Case may occasionally be seen in the wild. When used in a sentence, holidays should not be written in lowercase like any other word. AFAICT Yasumi provides stand-alone names, and for this purpose uppercasing the first letter of the first word is appropriate.
  • For consistency with Påskedag (Easter Day) and Pinsedag (Pentecost), I have changed 1. juledag to Juledag. All three days may be referred to as 1. xxxdag (literally _First Xxx Day) similar to 2. xxxdag, but I believe they are most commonly referred to with the 1..

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?

@c960657 c960657 force-pushed the denmark branch 2 times, most recently from 4fe2730 to af79350 Compare March 4, 2018 09:37
@stelgenhof stelgenhof self-requested a review March 4, 2018 12:06
@stelgenhof
Copy link
Member

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:

  • Official (i.e. public or national holidays)
  • Observance (Holidays that are celebrated but are generally a working day)
  • Seasonal (Holidays associated to seasonal events e.g. Summer Solstice)
  • Bank (Bank holiday)
  • Other (any other type of holiday)

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
Copy link
Contributor Author

c960657 commented Mar 4, 2018

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

@c960657 c960657 changed the base branch from master to develop March 6, 2018 07:43
@stelgenhof
Copy link
Member

@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
Copy link
Contributor Author

c960657 commented Apr 21, 2018

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

@stelgenhof stelgenhof added this to the v2.0.0 milestone Apr 23, 2018
@stelgenhof stelgenhof self-assigned this Apr 23, 2018
@stelgenhof stelgenhof merged commit b4073ce into azuyalabs:develop Apr 23, 2018
stelgenhof pushed a commit that referenced this pull request Jan 11, 2019
@c960657 c960657 deleted the denmark branch January 11, 2019 08:24
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.

2 participants