Skip to content

Conversation

@negro89
Copy link
Contributor

@negro89 negro89 commented May 29, 2025

Requerimiento

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

Funcionalidad desarrollada

  1. Se implementa un array para declarar permisos que deberían ser "incompatibles" (que no pueden estar activos simultáneamente) y poder hacer una verificación al activar un permiso. Este array podría implementarse dinámicamente en DB
  2. Se crean funciones para recorrer el arbol de permisos y dada una key poder encontrar el hijo correspondiente
  3. Evita request innecesarios segun permisos

UserStory llegó a completarse

  • Si
  • No
  • No corresponde

Requiere actualizaciones en la base de datos

  • Si
  • No

Requiere actualizaciones en la API

  • Si
  • No

Requiere actualizaciones en andes-test-integracion

  • Si
  • No

@negro89 negro89 requested review from a team as code owners May 29, 2025 14:34
@negro89 negro89 requested review from aldoEMatamala and ma7payne May 29, 2025 14:34
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.

Lauchis me pareció que quedó bastante bien la funcionalidad desde el gestor de usuarios, se comporta bien y es más intuitiva para el usuario.
Por otro lado, al aplicar los permisos desde la visualización de la HUDS estoy viendo algunos problemas que te paso a comentar:

  • Sin restricciones en la HUDS (huds completa), se cargan todos los laboratorios, pero no se ven los detalles de ninguno (SIL2 o CDAS).
  • Cuando tengo permiso restringido tampoco muestra todos los datos, y los laboratorios tardan mucho más en cargarse, por ejemplo probé:
    1. permiso por laboratorios y receta no se muestran los detalles de labos de CDAS.
    2. permiso por laboratorio, receta y vacunas muestra todos los laboratorios correctamente.

this.internacione$ = this.serviceMapaCamasHTTP.getPrestacionesInternacion(filtros);
if (this.permisoHudsCompleta) {
const filtros = {
fechaIngresoDesde: moment('2016-01-01').toDate(),
Copy link
Contributor

Choose a reason for hiding this comment

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

viendo esto, que ya estaba resuelto asi, por ahi podriamos pasarlo a la base ese dato? por ahi si no es entes pr podemos crear una tarea para no olvidarnos

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Me parece que es un dato que acá se setea muy excepcionalmente, no estoy seguro de llevarlo a la base. Podriamos evaluarlo. Mientras lo dejo como una constante de clase.

@negro89 negro89 force-pushed the HUDS-160 branch 3 times, most recently from 699dba0 to b75b52b Compare July 18, 2025 18:02
@negro89 negro89 requested a review from MCele July 18, 2025 18:02
@negro89
Copy link
Contributor Author

negro89 commented Jul 18, 2025

@MCele listos los cambios!

@MarianoCampetella
Copy link
Contributor

Cuando pongo los permisos para ver solamente los laboratorios, no puedo ver el detalle del mismo. Yo tengo cargado uno y es un CDA.
image

@MarianoCampetella MarianoCampetella added the changes requested Se solicitaron cambios label Aug 13, 2025
@Fabio-Ramirez
Copy link
Contributor

Exacatamente como menciona campe, no se ve la info, en cambio en demo si muestra dato despues de seleccionar el labo:
image

Lo demás estaría correcto.

@negro89
Copy link
Contributor Author

negro89 commented Aug 26, 2025

Exacatamente como menciona campe, no se ve la info, en cambio en demo si muestra dato despues de seleccionar el labo: image

Lo demás estaría correcto.

@MarianoCampetella @Fabio-Ramirez

Corregido y actualizado 👌🏼

@negro89 negro89 added changes done and removed changes requested Se solicitaron cambios labels Aug 26, 2025
Copy link
Contributor

@AgosLizzi AgosLizzi left a comment

Choose a reason for hiding this comment

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

Laucha, al poner permisos de visualización parcial de la HUDS (todos), ingresar a la huds y querer seleccionar labo, receta o vacunas me tira este error en la consola. Probé con HUDS completa e igual
error
image

@Fabio-Ramirez Fabio-Ramirez added changes requested Se solicitaron cambios and removed changes done labels Aug 27, 2025
@negro89
Copy link
Contributor Author

negro89 commented Aug 27, 2025

Laucha, al poner permisos de visualización parcial de la HUDS (todos), ingresar a la huds y querer seleccionar labo, receta o vacunas me tira este error en la consola. Probé con HUDS completa e igual error image

@AgosLizzi corregido!

@negro89 negro89 requested a review from AgosLizzi August 27, 2025 15:44
@negro89 negro89 added changes done and removed changes requested Se solicitaron cambios labels Aug 27, 2025
@Fabio-Ramirez
Copy link
Contributor

Fabio-Ramirez commented Sep 5, 2025

Compa, estarian bien las correcciones, ahora noto que se estarian haciendo request de mas, por ende se trae datos que no aplican. Por ejemplo, tengo permiso para visualizacion parcial, viendo el usuario Campetella Marian, luego se trae datos de sobra, algunos ejemplos...
image

image image image

@Fabio-Ramirez Fabio-Ramirez added changes requested Se solicitaron cambios and removed changes done labels Sep 5, 2025
@MarianoCampetella MarianoCampetella dismissed their stale review September 5, 2025 13:37

Se descarta ya que hay nuevas solicitudes de cambios.

@MCele MCele added the receta label Sep 9, 2025
@MCele MCele removed the receta label Oct 16, 2025
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.

5 participants