Skip to content

Comments

IN - Cambiar periodos de registros de internación#3115

Merged
negro89 merged 3 commits intomasterfrom
IN-612
Mar 26, 2025
Merged

IN - Cambiar periodos de registros de internación#3115
negro89 merged 3 commits intomasterfrom
IN-612

Conversation

@aldoEMatamala
Copy link
Contributor

@aldoEMatamala aldoEMatamala commented Nov 26, 2024

Requerimiento

https://proyectos.andes.gob.ar/browse/IN-612

Funcionalidad desarrollada

  1. Se reduce el rango inicial de registros de internación a 3 dias desde el día de la fecha
  2. Se modifica vigencia de tokenHuds de 4horas a 1hora
  3. Se modifica el pedido de motivo de acceso a la HUDS a sólo una vez hasta vencimiento de token
  4. Se agrega estructura de tokenHuds en memoria de sesion para que no se sobreescriba el token al querer revisar varios pacientes y se mantega el mismo token vigente

UserStory llegó a completarse

  • Si
  • No
  • No corresponde

Requiere actualizaciones en la base de datos

  • Si
  • No

Requiere actualizaciones en la API

Requiere actualizaciones en andes-test-integracion

  • Si
  • No

@Fabio-Ramirez Fabio-Ramirez added the changes requested Se solicitaron cambios label Dec 17, 2024
@aldoEMatamala aldoEMatamala added changes done and removed changes requested Se solicitaron cambios labels Jan 9, 2025
Copy link
Contributor

@negro89 negro89 left a comment

Choose a reason for hiding this comment

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

Compa, el filtro de fechas para los registros de la internacion quedó de 10 pero hay un temita con el modal de motivo de acceso: Por un lado, despues de varias pruebas me sigue pidiendo el motivo por cada vez que quiero ver los registros (acepto el modal, me muevo al tab "cama" y despues de nuevo a "internacion -> registros" y vuelve a pedir el motivo (yo supongo que debe estar molestando el behaviorSubject que agregaste). Por otro lado, si selecciono motivo y acepto y despues selecciono una cama sin paciente me vuelve a pedir el motivo (en demo actualmente no pasa). Dejo videito: https://www.loom.com/share/1902ad83dbb440f2860a81a32ccf0067?sid=d8847647-4144-42e6-9998-25eae1c7800e

La solución ideal seria almacenar un array con los ultimos motivos de acceso (hoy en dia solo se guarda el ultimo, es decir, cada vez que aceptamos el modal el motivo anterior se pisa) y cada vez que se solicite acceso a la huds revisar si en este array existe un token aún vigente que contenga al usuario logeado, paciente y organizacion. En caso de que exista se lo utiliza en lugar de generar un token nuevo.

Para esto deberían suceder las siguientes cosas:

  1. los tokens deberían generarse con una validez de 1 hora
  2. el array de tokens deberá tener un maximo de 15 elementos
  3. los tokens vencidos deben borrarse del array

@aldoEMatamala aldoEMatamala force-pushed the IN-612 branch 2 times, most recently from 5065696 to ba986bc Compare February 13, 2025 18:43
@aldoEMatamala
Copy link
Contributor Author

Compa, el filtro de fechas para los registros de la internacion quedó de 10 pero hay un temita con el modal de motivo de acceso: Por un lado, despues de varias pruebas me sigue pidiendo el motivo por cada vez que quiero ver los registros (acepto el modal, me muevo al tab "cama" y despues de nuevo a "internacion -> registros" y vuelve a pedir el motivo (yo supongo que debe estar molestando el behaviorSubject que agregaste). Por otro lado, si selecciono motivo y acepto y despues selecciono una cama sin paciente me vuelve a pedir el motivo (en demo actualmente no pasa). Dejo videito: https://www.loom.com/share/1902ad83dbb440f2860a81a32ccf0067?sid=d8847647-4144-42e6-9998-25eae1c7800e

La solución ideal seria almacenar un array con los ultimos motivos de acceso (hoy en dia solo se guarda el ultimo, es decir, cada vez que aceptamos el modal el motivo anterior se pisa) y cada vez que se solicite acceso a la huds revisar si en este array existe un token aún vigente que contenga al usuario logeado, paciente y organizacion. En caso de que exista se lo utiliza en lugar de generar un token nuevo.

Para esto deberían suceder las siguientes cosas:

  1. los tokens deberían generarse con una validez de 1 hora
  2. el array de tokens deberá tener un maximo de 15 elementos
  3. los tokens vencidos deben borrarse del array

Hola lauta! gracias por los comentarios, quedan los tokenHuds en arreglo para que no vuelva a pedir, y queda la vigencia de 1hora
image

@negro89 negro89 merged commit 2d86948 into master Mar 26, 2025
2 checks passed
@negro89 negro89 deleted the IN-612 branch March 26, 2025 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants