-
-
Notifications
You must be signed in to change notification settings - Fork 197
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
Indian National Holidays #12
Conversation
@Nielsvanpach attempted to add region support, can you review |
@RohanSakhale Most of the holidays you have added will change every year, eg Diwali will be on a different date each year. |
@lokesh-zersys yes that is true, that thought was in my mind, but need to think of a way out to provide dates based on Hindu Calendar, which I was searching but couldn't found concreate way out unless we start defining them ourself in the repo |
In my opinion, we should add only Mandatory Holidays based on Country's and respective State's(Region) Law. For e.g. according to Maharastra Law. There are only 4. 3 National Holiday
1 State Holiday
|
Also, list of holidays changes based on sector as well, not just region. E.g. banking and school sector will have more holidays than offices. Just a thought to keep in mind when creating a list of Holidays that is generic for all. |
I've added documentation on how I think we should manage regions or states: Could you get this PR up to date or maybe better, start a new one? |
@Nielsvanpach updated based on documented structure I believe we should also support |
'Ram Navami' => '04-21', | ||
'Mahavir Jayanti' => '04-25', | ||
'Buddha Purnima' => '05-26', | ||
'Eid al-Fitr' => '05-14', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think a fixed list can be used for Islamic holidays
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There we would need a manual intervention for adding Islamic and Hindu holidays based on Moon based calendar, this can either come through config or via class abstract methods
Thanks, but I'm closing this PR as it doesn't feel complete yet. Feel free to reopen once fixed! |
This PR introduces national holidays in India referenced from Wiki Public Holidays Page
New Changes
To ask
Many times there are state-specific holidays in many countries, we can optionally introduce state-specific holidays that can be passed as active state variables or variables when checking
isHoliday
this would make the repository very useful when utilizing it across countries.