HUDS - Carga de matrícula

[aldoEMatamala]

Requerimiento

https://proyectos.andes.gob.ar/browse/HUDS-129

Funcionalidad desarrollada

1. Se guarda la primer matrícula VIGENTE que encuentre
2. En caso de tener todas vencidas se guarda la última matrícula actualizada

UserStories llegó a completarse

• Si
• No

Requiere actualizaciones en la base de datos

• Si
• No

[agustin1996ra]

Para mi esta bien

[MarianoCampetella]

Funciona bastante bien pero hay un par de cositas que se podrían mejorar. Ya que estamos utilizando moment() para trabajar con las fechas entonces remplazar los new Date() por los moment (en todo caso se puede utilizar moment().toDate()). También se podría remplazar el any al momento de declarar la variable _profesional de la linea 156 por la interfaz IProfesional que existe (seguramente tengas que completar dicha interfaz ya que tiene muy pocos atributos).

[aldoEMatamala]

Funciona bastante bien pero hay un par de cositas que se podrían mejorar. Ya que estamos utilizando moment() para trabajar con las fechas entonces remplazar los new Date() por los moment (en todo caso se puede utilizar moment().toDate()). También se podría remplazar el any al momento de declarar la variable _profesional de la linea 156 por la interfaz IProfesional que existe (seguramente tengas que completar dicha interfaz ya que tiene muy pocos atributos).

hola campe, gracias por las observaciones!
Se cambio a moment todos los Date y se agregaron como atributos opcionales a la interfaz IProfesional para el correcto casteo.

[aldoEMatamala]

Se prueba nuevamente la funcionalidad ok
demo vs PR

[aldoEMatamala]

Actualización:
Se corrigen tipos /variables sin uso/ flujo/ eficiencia en comparación