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 Andalusia region (AN) holidays #56

Closed
wants to merge 12 commits into from

Conversation

vice
Copy link
Contributor

@vice vice commented Jun 20, 2020

Some of these holidays could be celebrated in others regions of Spain. At least they are correct for Andalusia region (AN).

vice added 9 commits June 20, 2020 11:02
Signed-off-by: Vicente Jimenez Aguilar <googuy@gmail.com>
Signed-off-by: Vicente Jimenez Aguilar <googuy@gmail.com>
when Epifanía del Señor (01-06) is sunday

Signed-off-by: Vicente Jimenez Aguilar <googuy@gmail.com>
when Inmaculada Conceptción is on sunday

Signed-off-by: Vicente Jimenez Aguilar <googuy@gmail.com>
Signed-off-by: Vicente Jimenez Aguilar <googuy@gmail.com>
when national Fiesta del Trabajo (05-01) holiday is on Sunday

Signed-off-by: Vicente Jimenez Aguilar <googuy@gmail.com>
Signed-off-by: Vicente Jimenez Aguilar <googuy@gmail.com>
when national Navidad holiday is on Sunday

Signed-off-by: Vicente Jimenez Aguilar <googuy@gmail.com>
when Andalucia's day (02-28) is Sunday

Signed-off-by: Vicente Jimenez Aguilar <googuy@gmail.com>
@lauft
Copy link
Member

lauft commented Jun 20, 2020

Thanks for your contribution! 👍
Can you provide the online sources which define these holidays (official legal documents preferred). This will help to check on updates in the future.

@vice
Copy link
Contributor Author

vice commented Jun 20, 2020

I used those references
https://administracion.gob.es/pag_Home/atencionCiudadana/calendarios/laboral.html
http://www.seg-social.es/wps/portal/wss/internet/CalendarioLaboral
https://es.wikipedia.org/wiki/Calendario_laboral (Not official but accurate)

For Andalusia
https://www.juntadeandalucia.es/temas/trabajar/relaciones/calendario.html

Also checked https://www.calendarioslaborales.com/ they could be used to check holidays in other Spanish regions and locations but is not an official source and could contain errors.

@vice
Copy link
Contributor Author

vice commented Jun 20, 2020

From the first reference (https://administracion.gob.es/pag_Home/atencionCiudadana/calendarios/laboral.html) you can find this legal document https://www.boe.es/diario_boe/txt.php?id=BOE-A-2019-14552
But this is just for 2020. And there is no reference to 2 local holidays because they are decided locally. Holidays in Spain is a mess. See #57 for a suggestion.

@vice
Copy link
Contributor Author

vice commented Jun 20, 2020

In Spain is very common to have Monday holidays when a fixed one is on Sunday but it is a regional decision. I added those that Andalusia region had done for many years (2006 - 2020) as far as I have researched. I think generally others regions could be included but they need to be checked one by one.

Maybe is better to assume that the Sunday holiday produced a Monday holiday and make correction when we discover this is not true for any region. I am open to suggestions.

@vice
Copy link
Contributor Author

vice commented Jun 20, 2020

I probably will do more addition and correction from official documents. Including for other regions besides Andalusia. I need to change branch name as my first intention was just to add regional holidays as in commit e8c69ce.

Signed-off-by: Vicente Jimenez Aguilar <googuy@gmail.com>
01-06 is know as 'Día de Reyes' but this is an informal name,
      officially the name is 'Epifanía del Señor'
11-01 'Día de Todos los Santos' or 'Festividad de Todos los Santos'
      both refer to 'Todos los Santos' day or holiday
12-06 'Día de la Constitución Española' the official name is explicit
      because in Spain we have regional constitutions also
Reference:
https://www.boe.es/boe/dias/2019/10/11/pdfs/BOE-A-2019-14552.pdf

Signed-off-by: Vicente Jimenez Aguilar <googuy@gmail.com>
@vice
Copy link
Contributor Author

vice commented Jun 20, 2020

I think I pushed a change in the middle of the checks. I will check logs later. I need to leave.

It is customary to stop working early even at noon
to let people have dinner with their families or friends
But they are not official holidays

Signed-off-by: Vicente Jimenez Aguilar <googuy@gmail.com>
@vice
Copy link
Contributor Author

vice commented Jun 20, 2020

I see, I need to check tests because I changed names and remove some days. I will do later.

@lauft
Copy link
Member

lauft commented Jun 20, 2020

I see, I need to check tests because I changed names and remove some days. I will do later.

You most probably will need to update the snapshots by calling

pytest test/test_holidata.py --snapshot-update

@vice
Copy link
Contributor Author

vice commented Jun 21, 2020

Thanks for the tip. I'm going to close this pull request a prepare a new one updating code and tests snapshots.

@vice vice closed this Jun 21, 2020
@vice vice deleted the Andalucía branch June 21, 2020 22:03
@lauft lauft added the locale label Sep 2, 2021
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.

None yet

2 participants