Skip to content

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

Open
JuanIRamirez wants to merge 2 commits into
masterfrom
MONIT-60
Open

MONIT-60: ABM de restricción por paciente#1868
JuanIRamirez wants to merge 2 commits into
masterfrom
MONIT-60

Conversation

@JuanIRamirez

@JuanIRamirez JuanIRamirez commented Feb 9, 2024

Copy link
Copy Markdown
Contributor

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

MCele commented Jul 19, 2024

Copy link
Copy Markdown
Contributor

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

@MCele MCele left a comment

Copy link
Copy Markdown
Contributor

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.

Comment thread auth/schemas/authUsers.ts Outdated
idPaciente: string;
archivos: [any];
observaciones: string;
createdBy: any;

@MCele MCele Jul 19, 2024

Copy link
Copy Markdown
Contributor

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
@silviroa silviroa closed this Apr 7, 2026
@MCele MCele reopened this Aug 21, 2026
@MarianoCampetella

Copy link
Copy Markdown
Contributor

A la fecha de hoy se actualiza pr con todas las correcciones solicitadas. Estoy atento a nuevos cambios.
Saludos!

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.

5 participants