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

MONIT-60: ABM de restricción por paciente #1868

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

JuanIRamirez
Copy link
Contributor

@JuanIRamirez JuanIRamirez commented Feb 9, 2024

Requerimiento

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

Funcionalidad desarrollada

UserStories llegó a completarse

  • Si
  • No

Requiere actualizaciones en la base de datos

  • Si
  • No

@MCele
Copy link
Contributor

MCele commented Jul 19, 2024

La funcionalidad desde monitoreo se ve joya. Pero me parece que falta pulir algunas cositas:

  • En la tarea dice: "Esta información se guardará dentro del campo "permisosGlobales" de la colección authUsers, incluyendo el Audit del usuario que creó/actualizó dicha restricción. El nombre del campo será un array llamado "pacienteRestringido"."
    No se está guardando dentro de permisosGLobales.
  • Los datos del usuario que creó/modificó la restricción no se están guardando, si no que se guarda información del mismo usuario que se está restringiendo, además la fecha y hora de este cambio siempre se guarda sobre "createdAt", aunque debería ser "updatedAt" cuando se lo modifica. NOTA: Se puede ver en la imagen que los datos del usuario están incompletos.
  • La ruta de acceso a los archivos adjuntos (atrubuto url) es raro que se guarde de forma absoluta, yo optaría que se guarde un id vinculado (igual esto último me pasó la primera vez y luego ya no pude volver a reproducirlo, así que si no pasa lo desestimamos).
    Por otro lado, habría que revisar si es conveniente guardar los archivos en la bd o recurrimos al drive.
    Con respecto a este tema, si sigue guardándose en la bd,
    image

Por ahí respecto a información de los archivos, consideraría lo siguiente sólo como sugerencia, para mantener el mismo mecanismo de guardado de datos:
Mantener datos del archivo, como formato y demás sólo en la colección de RestriccionHUDSStore.files y si existen archivos ahí, levantarlos en el listado. Es decir, que la información faltaría guardarse en el atributo "metadata" de la colección sería idUser, idPaciente restringido, incluso evitaría tener que guardar el dato de formato de archivo y demás detalles en el usuario. Esto último es una apreciación personal, considerando que si necesitamos saber a qué usuario pertenece cada elemento registrado la búsqueda se va a hacer imposible desde la colección users si esto creciera

Copy link
Contributor

@MCele MCele left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Como detalle visual, (más desde el lado de monitoreo app), faltan ver datos del profesional asociado. Veo que sólo muestra si está matriculado o no.
Agregaría a lo ya comentado antes, analizar si el profesional a vincular al usuario debe considerarse que esté habilitado o no, actualmente sólo trae el profesional si está habilitado, pero si no está, no lo vincula.

idPaciente: string;
archivos: [any];
observaciones: string;
createdBy: any;
Copy link
Contributor

@MCele MCele Jul 19, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

al ser de tipo any está guardando solo documento y del usuario restringido, no del que crea la restricción o la actualiza. No sería mejor para created o updated usar AuditPlugin?

@github-actions github-actions bot removed the has_conflicts Tiene conlfictos label Jul 30, 2024
@github-actions github-actions bot added the has_conflicts Tiene conlfictos label Jan 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
has_conflicts Tiene conlfictos
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants