MONIT - Dar de baja pacienteApp

[negro89]

Requerimiento

https://proyectos.andes.gob.ar/browse/MONIT-47

Funcionalidad desarrollada

1. Nuevo atributo baja en esquema de pacienteApp.
2. Nueva ruta patch en 'mobileApp/paciente' para dar de baja una cuenta mobile.
3. Filtro en recurso de pacienteApp para ignorar los registros con atributo baja: true

UserStories llegó a completarse

• Si
• No

Requiere actualizaciones en la base de datos

• Si
• No

[maring0019]

• Falta probar router.patch('/account/baja/:id', async (req: any, res, next) (ver archivo modules/mobileApp/routes/accounts.ts) para ver si funciona la captura de errores.

• Correr los test para ver si es necesario crear o modificar test.

[MarianoCampetella]

Si bien cumple con los requisitos de la tarea, se podrían hacer algunas mejoras dentro del código. Dentro de "autenticacionApp.ts" en el POST "/olvide-password" existen dos if que retornan el mismo mensaje, el cual se podría simplificar en uno solo.
También se observa que cuando no existe el paciente, la variable datosUsuario se transforma en null con lo cual en el segundo if ubicado en la linea 100 va a generar un error al acceder a una variable null.
Si no es muy complicado estaría bueno agregar alguna condición de buscar al paciente a través del email y fijarse si esta dado de baja o no con la nueva variable que se agrego. Si esto ocurre se podría imprimir un mensaje al usuario para informarle de la situación ya que actualmente me permite enviar un código de activación aunque se encuentre de baja. Revisar mismo comportamiento en las otras partes del código!