-
Notifications
You must be signed in to change notification settings - Fork 122
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
"Bridge" holidays #67
Comments
There's no built-in way to do that. You should be able to create a custom HolidayFn that returns Friday or Tuesday after checking if the original holiday is on Thursday or Tuesday. |
Thank you @rickar! If I understand correctly, that HolidayFn would swap the "Thursday" with "Friday", but in my case, we need to add Friday, and finally should be two holidays: Thursday and Friday. |
Right. Each Holiday definition can only return one day per year so you would need to create something like TruthDay and SecondTruthDay. TruthDay would return the normal holiday every year like you currently have it with CalcDayOfMonth and SecondTruthDay would use a custom CalcBridgeDay that returns the extra day if it applies or zero time if not. |
I see. So I should define this "second" day for each actual holiday...
El sáb., 29 de mayo de 2021 10:58, rickar ***@***.***>
escribió:
… Right. Each Holiday definition can only return one day per year so you
would need to create something like TruthDay and SecondTruthDay. TruthDay
would return the normal holiday every year like you currently have it with
CalcDayOfMonth and SecondTruthDay would use a custom CalcBridgeDay that
returns the extra day if it applies or zero time if not.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#67 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAJZGPJ2X6G7G4GPQQB5ITTQDXHLANCNFSM45LWNVIQ>
.
|
Hello, here in Argentina, we have a rule for promote turism, which adds Friday as holiday when the real one is Thursday, and adds Monday when it is Tuesday.
Is there any way to represent it with the new v2 api?
Thanks!
The text was updated successfully, but these errors were encountered: