date-holidays icon indicating copy to clipboard operation
date-holidays copied to clipboard

Added Nation-Wide and State-Wide Holidays in India as per Gregorian Calendar

Open DrunkenCloud opened this issue 8 months ago • 8 comments

Many holidays follow their own calendars like the Hindu or Islam calendar. States as well have more authority to choose holidays so nation wide holidays are limited.

Issue: https://github.com/commenthol/date-holidays/issues/527

DrunkenCloud avatar Apr 25 '25 08:04 DrunkenCloud

Added a fix. Apologies for not seeing it earlier

DrunkenCloud avatar Apr 27 '25 01:04 DrunkenCloud

Nice to see to support for India added. Great work! I wonder how could we add support for dynamic holidays like festivals in India. I feel the directory structure should look like this

country -> {name} -> constant_holidays -> dynamic_holidays

dynamic_holidays might need to be updated start of every year.

ranjith-gampa avatar Apr 30 '25 17:04 ranjith-gampa

Yeah that should work, I suppose the parsing will change, not that much so yeah. Currently its like IN.yaml, and if im not wrong, your proposing

countries/ IN/ |---- constant.yaml // constant holidays |---- dynamic.yaml // dynamic holidays

for each country? keeping the current file structure same. There are some opensource tools that give festivities dates, so I suppose we will be using those dates for each year.

Shall I start working on that in a new branch?

DrunkenCloud avatar May 01 '25 09:05 DrunkenCloud

I just noticed that languages support in the file was wrong, fixed that. There are some spelling mistakes in the tamil names, I did that on purpose due to some unicode problems, I dont understand why, it doesnt render properly, so I rewrote it to work (My mother tongue is Tamil)

DrunkenCloud avatar May 01 '25 10:05 DrunkenCloud

Glad to see the support for Indian calendar, although I see Good Friday is missing from the list of holidays.

easter -2:
  name:
    en: Good Friday
    hi: गुड फ्राइडे
    ta: புனித வெள்ளி

hrishabh-kumar avatar May 02 '25 05:05 hrishabh-kumar

Thanks for noticing. Added it!

DrunkenCloud avatar May 02 '25 05:05 DrunkenCloud

Hi @DrunkenCloud, The "regions:" property is not known. Can you please check the specification.md?

commenthol avatar May 12 '25 11:05 commenthol

On it. Im currently changing the format.

DrunkenCloud avatar May 12 '25 11:05 DrunkenCloud

@commenthol is there a plan to merge this PR ?

gethari avatar Aug 18 '25 04:08 gethari

Hi @gethari, this PR does not pass CI and needs to get fixed first.

commenthol avatar Aug 23 '25 12:08 commenthol

@DrunkenCloud can you fix the CI job failures

gethari avatar Aug 29 '25 07:08 gethari

working on it. Adding fixtures in the test folder

DrunkenCloud avatar Aug 31 '25 06:08 DrunkenCloud