denmark: General prayer day only until 2023 about jollyday HOT 7 CLOSED

Yeah, your absolutely correct @derTobsch , my apologies

from jollyday.

Thanks for your issue @AndreasPetersen,

as described here https://en.wikipedia.org/wiki/Public_holidays_in_Denmark the 24.12. is not a official public holiday. Is that correct? Same here https://github.com/nager/Nager.Date/blob/main/src/Nager.Date/PublicHolidays/DenmarkProvider.cs.

from jollyday.

Isn't there still the following issues?

• names of the holidays on December 25 and 26 are incorrect. I would argue that the English Wikipedia also has the incorrect names. In Danish they are litterally called first and second christmas day (perhaps with the first it is often just called christmas day). I can see that there is a first and second Christmas day name

  jollyday/jollyday-core/src/main/resources/descriptions/holiday_descriptions.properties

  Lines 306 to 307 in eb88454

  	holiday.description.FIRST_CHRISTMAS_DAY = 1st Christmas day
  	holiday.description.SECOND_CHRISTMAS_DAY = 2nd Christmas day

• General prayer day is no longer starting next year (should have a validTo="2023" added)

from jollyday.

If you want you can provide a pull request and improve the public holidays of Denmark. Of course you will get support from me.

from jollyday.

Hey @AndreasPetersen,

I removed the general payers day starting from 2024.

I have no idea how I will continue with the translations in this lib. It is possible to add a new holiday_descriptions_dk.properties to provide all the translation needed. But on the other side, we need to add all translations for all public holidays provided by jollyday, even if this public holiday is not available in Denmark.

Maybe we should provide a global holiday_descriptions.properties with the English names of the public holidays for all holidays and provide specific holiday_descriptions_dk.properties that only overrides the public holidays in Denmark. What do you think?

from jollyday.

I extracted the naming issue into #201 so we can fix the general prayer day in this issue

from jollyday.

Hi @derTobsch ,

Thank you for the effort, and apologies I haven't replied. I think it makes sense to extract the naming to a separate issue as done. I think you suggestion makes sense.

from jollyday.