search

pjnovas / nolaborables

API pública y gratuita para consultar los feriados en Argentina.
http://nolaborables.com.ar/
BSD 2-Clause "Simplified" License
108 stars 21 forks source link

Reportar nombre del día ademas de numero de dia y de mes en /feriados/2019 #18

Open Ambro17 opened 5 years ago

Ambro17 commented 5 years ago

En el uso de la api me encontré manualmente construyendo días a partir del numero de dia, mes y año. Tal vez se podría agregar el nombre del día, o algún atributo con la fecha cruda (isoformat) tal vez para facilitar el manejo de la fecha. El cambio sería de:

[{'motivo': 'Año Nuevo',
  'tipo': 'inamovible',
  'dia': 1,
  'mes': 1,
  'id': 'año-nuevo'},
   ...
]

a

[{'motivo': 'Año Nuevo',
  'tipo': 'inamovible',
  'dia': 1,
  'mes': 1,
  'nombre_dia': Lunes
  'id': 'año-nuevo'},
   ...
]

De esta forma es mas facil ignorar los feriados que caen en fines de semana por ejemplo. Por supuesto que es un enhancement y nada indispensable pero estaría muy piola si lo tuviera.

pjnovas commented 5 years ago

sep, suena copado pero lo haría mas con un índice de día de la semana (enumerador), el nombre "lunes" lo dejaría del lado del consumidor.

Agregar la fecha en ISO suena bastante piola ya que se podría utilizar para construir la fecha directamente desde cualquier lenguaje, quedaría algo asi:

{
  "tipo": "inamovible",
  "dia": 1,
  "mes": 1,
  "id": "año-nuevo",
  "diaSemana": 2,
  "iso": "2019-01-01T00:00:00.000Z"
}

En ese caso se podría filtrar todos los "diaSemana" que sean 0 (domingo) o 6 (sábado).

Ambro17 commented 5 years ago

Si, totalmente de acuerdo, el dia numerico sería más compatible. No sé si agregaría la hora en iso con la timezone en UTC. Creo que correspondería -03:00.

pjnovas commented 5 years ago

Si, tenes razón, son feriados Argentinos, la hora debería estar en -3.

Cuando tenga un rato lo armo, no lo veo complejo, gracias dude