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

Add locale et-EE #81

Merged
merged 1 commit into from
Jan 14, 2022
Merged

Add locale et-EE #81

merged 1 commit into from
Jan 14, 2022

Conversation

lauft
Copy link
Member

@lauft lauft commented Jan 12, 2022

This adds the locale et-EE (Estonia) according to the Public Holidays and Days of National Importance Act
(Pühade ja tähtpäevade seadus)

Before merging, the following has to be resolved:

  • Have there been any changes to the listed holidays since 2011?
  • Ülestõusmispühade 1. püha refers to Easter Sunday
  • Nelipühade refers to Pentecost Sunday

@lauft lauft mentioned this pull request Jan 12, 2022
@lauft lauft self-assigned this Jan 12, 2022
@lauft lauft added help wanted Extra attention is needed locale labels Jan 12, 2022
@h0adp0re
Copy link

h0adp0re commented Jan 13, 2022

Before merging, the following has to be resolved:

*  Have there been any changes to the listed holidays since 2011?

*  Ülestõusmispühade 1. püha refers to Easter Sunday

*  Nelipühade refers to Pentecost Sunday
  • The current state of the law can be seen if you press the "In force" button on the right hand side in the article header. I gave it a once-over and everything seems to be in order.
  • Correct, Ülestõusmispühade 1. püha is Easter Sunday.
  • Correct, Nelipühade 1. püha is Pentecost Sunday, a.k.a. Whit Sunday a.k.a. Whitsun. I've commented on the correct name separately in the diff.

Thanks for taking the time, I appreciate it!

holidata/holidays/et-EE.py Outdated Show resolved Hide resolved
holidata/holidays/et-EE.py Outdated Show resolved Hide resolved
holidata/holidays/et-EE.py Outdated Show resolved Hide resolved
@h0adp0re
Copy link

I looked at the German holidata file and realized you probably have some days which are legally designated half-workdays? Am I right? If so, can you please add such days here as well? I will provide data.

@lauft
Copy link
Member Author

lauft commented Jan 13, 2022

I looked at the German holidata file and realized you probably have some days which are legally designated half-workdays? Am I right? If so, can you please add such days here as well? I will provide data.

Can you tell me to which days you are referring to? Currently I am not aware that there are legally defined half-holidays in Germany. There are some days, where it is custom to stop work early in the afternoon/after lunch, but this is not legally binding and depends on your employer in my experience.

Holidata tries to focus on the "holidays defined by law" as the smallest common ground, therefore I would rather remove such holidays from the database... but this depends on the specific case.

@h0adp0re
Copy link

Can you tell me to which days you are referring to? Currently I am not aware that there are legally defined half-holidays in Germany. There are some days, where it is custom to stop work early in the afternoon/after lunch, but this is not legally binding and depends on your employer in my experience.

I'm sorry, it's possible I am mistaken, I merely skimmed the file. Maybe the act of skimming it reminded me that such things exist.

Holidata tries to focus on the "holidays defined by law" as the smallest common ground, therefore I would rather remove such holidays from the database... but this depends on the specific case.

That's understandable. There are 4 dates defined in our law that are to be 3 hours shorter workdays. This applies to all workers who work according to the standard business schedule, but not to workers who have a variable schedule as per their contract.

I refer you to § 53. of our Employment Contracts Act – Shortening of working time:

An employer shall shorten the working day preceding New Year’s Day, the anniversary of the Republic of Estonia, Victory Day and Christmas Eve by three hours.

@lauft
Copy link
Member Author

lauft commented Jan 13, 2022

I refer you to § 53. of our Employment Contracts Act – Shortening of working time

There is #36 which might be also applicable to this case, but here it seems that only the mandatory work hours on those days have to be reduced by 3 hours, right? So there are some employees who might start earlier than others?

Until #36 is resolved, I would leave those cases out...

@lauft
Copy link
Member Author

lauft commented Jan 13, 2022

The current state of the law can be seen if you press the "In force" button on the right hand side in the article header. I gave it a once-over and everything seems to be in order.

Holidata strives to deliver correct data from 2011 on (of course the main focus is on the current/coming year).

I tried to check the laws from previous years and did not spot any differences, but the process of selection of the different versions/years was not really clear to me and I hoped the original version would be a bit clearer.

@lauft
Copy link
Member Author

lauft commented Jan 13, 2022

I updated the PR according to your review comments. If there haven't been any changes to holidays since 2011, we are good to go. 🚀

@h0adp0re
Copy link

There is #36 which might be also applicable to this case, but here it seems that only the mandatory work hours on those days have to be reduced by 3 hours, right?

Yes, mandatory hours are reduced.

So there are some employees who might start earlier than others?

Yes, if your employment contract specifies non-standard working hours, the law might not apply to you, since those reduced hours are discounted from elsewhere. It is a bit confusing but that's non-standard for ya...

Otherwise no, I don't think anyone starts earlier just to still work 8 hours and get off work at the same time as their colleagues. If the law applies to you, your workday is shortened by 3 hours.

Until #36 is resolved, I would leave those cases out...

Yes, that is acceptable. It's a bit of an edge case anyway.

@h0adp0re
Copy link

I tried to check the laws from previous years and did not spot any differences, but the process of selection of the different versions/years was not really clear to me and I hoped the original version would be a bit clearer.

It can be a bit confusing, yes. I find it to be easiest with the "Compare wordings" button.

Between 2011 and now the public holidays haven't changed, so we're good there.

There have been a couple of changes in days of national importance but since those aren't needed in this project, we're good there as well.

I updated the PR according to your review comments. If there haven't been any changes to holidays since 2011, we are good to go. 🚀

Then we are good to go, looks good!

LGTM and thank you for your time!

@lauft
Copy link
Member Author

lauft commented Jan 13, 2022

LGTM and thank you for your time!

Thank you for your time as well. 👍🏻
Hope you keep can an eye on those laws and give a head up in case something changes. ❤️

@h0adp0re
Copy link

LGTM and thank you for your time!

Thank you for your time as well. 👍🏻 Hope you keep can an eye on those laws and give a head up in case something changes. ❤️

I will do my best!

Closes #81
Closes #47

Signed-off-by: Thomas Lauf <[email protected]>
@lauft lauft removed the help wanted Extra attention is needed label Jan 14, 2022
@lauft lauft merged commit 1656dcd into master Jan 14, 2022
@lauft lauft deleted the locale/et-EE branch January 14, 2022 14:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants