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

Consistencia con sitio del ministerio - feriados 2019 #16

Closed
Ambro17 opened this issue Dec 15, 2018 · 1 comment
Closed

Consistencia con sitio del ministerio - feriados 2019 #16

Ambro17 opened this issue Dec 15, 2018 · 1 comment

Comments

@Ambro17
Copy link
Contributor

Ambro17 commented Dec 15, 2018

Hola, primero que nada felicitaciones por este desarrollo. Espero que el proyecto siga vivo.

Llegué acá porque actualmente estoy parseando el sitio https://www.argentina.gob.ar/interior/feriados para obtener los feriados, pero le pusieron una comprobación extra que exige que el cliente tenga habilitado js que rompió mi script, con lo cual me parece buena idea empezar a usar esta api y consumir datos en un formato más consistente.

A la hora de comparar los resultados de la API con los del sitio del ministerio encontré algunas -pocas- diferencias. A saber:
17-06-2019. En el ministerio figura trasladable. La api lo devuelve como nolaborable
31-08-2019. En el ministerio figura no laborable por 'Año Nuevo Islámico'. La api no lo tiene registrado como feriado ese día sino el 11-09-2019. El día correcto según una rápida búsqueda en google es el 31-08-2019, como dice el sitio del ministerio.

Gracias de antemano, no tengo problema en agregarlos yo si me explicás el proceso o de donde parseas los datos.
Saludos o/

@Ambro17 Ambro17 changed the title Actualizar feriados 2018 y 2019 Consistencia con sitio del ministerio - feriados 2019 Dec 15, 2018
pjnovas pushed a commit that referenced this issue Dec 15, 2018
…riado 'martin-guemes' (#17)

* Fix islamic new year to proper date of 2019
* Modify martin-guemes from nolaborable into trasladable
@pjnovas
Copy link
Owner

pjnovas commented Dec 15, 2018

@Ambro17 gracias por reportar los errores y por el PR!, ya estan deployados los cambios. saludos!

@pjnovas pjnovas closed this as completed Dec 15, 2018
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

No branches or pull requests

2 participants