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

How to override holidays #116

Closed
MathijsK93 opened this issue May 4, 2015 · 3 comments · Fixed by #126
Closed

How to override holidays #116

MathijsK93 opened this issue May 4, 2015 · 3 comments · Fixed by #126

Comments

@MathijsK93
Copy link
Contributor

We're using the holidays gem to prevent performing actions on holidays, but some holidays defined in the yaml files, such as liberation day, is an normal working day.

I know how to add custom holidays, but how can i override a specific holiday?

@espen
Copy link
Contributor

espen commented May 6, 2015

If the day is a working day in your country it should be listed as informal in the data files, right? If you can document this is correct then I suggest you send a pull request.

@ppeble
Copy link
Member

ppeble commented May 9, 2015

@MathijsK93 Yep, @espen had the right of it. If you have documentation (wikipedia, some government site, etc) that the definition is wrong then we can discuss it or you can submit a PR with the update. That would be the best course of action. So, for example, if the day you specified is a holiday but not observed as a day off then we should probably update the defs to make it 'informal'.

As for overriding I don't believe we have the ability to do that. We assume that the definitions represent truth. I'll see what happens when you try to load a 'custom' holiday for an existing date. I've never personally tried.

@MathijsK93
Copy link
Contributor Author

I created an Pull request for this: #126

Murphydbuffalo pushed a commit to Murphydbuffalo/holidays that referenced this issue Sep 21, 2021
Add Berlin's new International Women's Day
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants