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

HUDS - Controla permisos para ver la HUDS de pacientes #2978

Merged
merged 2 commits into from
Jul 18, 2024
Merged

Conversation

ma7payne
Copy link
Contributor

@ma7payne ma7payne commented Feb 19, 2024

Requerimiento

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

Funcionalidad desarrollada

  1. Crea un PacienteRestringidoPipe para verificar restricciones de un profesional sobre un paciente y las aplica en los módulos indicados
  2. Chequea restricción en coleccion auhtUsers: pacienteRestringido

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

RELACIONADO CON:
andes/api#1868
andes/monitoreo-app#68

@ma7payne ma7payne requested review from a team as code owners February 19, 2024 18:47
@ma7payne ma7payne requested review from MCele and AgosLizzi February 19, 2024 18:47
@ma7payne ma7payne force-pushed the HUDS-88 branch 2 times, most recently from 2976844 to 91032d7 Compare April 18, 2024 17:58
@negro89
Copy link
Contributor

negro89 commented May 7, 2024

Mati, funciona de 10. Lo unico que habría que retocar es una restricción en RUP para evitar que el usuario pueda ..

  1. Iniciar prestaciones CON TURNO
  2. Iniciar prestaciones fuera de agenda
  3. Agregar sobreturnos del paciente restringido

Por ejemplo, para una restriccion hacia mi sobre el paciente matamala aldo en la imagen 1 no debería poder iniciar prestación ni ver huds, mientras que en la imagen 2 debería lanzarse un cartel invasivo advirtiendo que no tengo pemisos para iniciar prestaciones sobre ese paciente. Para el caso de la imagen 3 debebría funcionar igual que en la 2.

IMAGEN 1:
image

IMAGEN 2:
image

IMAGEN 3:
image

@negro89 negro89 added the changes requested Se solicitaron cambios label May 8, 2024
@ma7payne ma7payne force-pushed the HUDS-88 branch 17 times, most recently from d61f2f4 to 6dd8d11 Compare June 18, 2024 15:53
@ma7payne ma7payne added changes done and removed changes requested Se solicitaron cambios labels Jun 18, 2024
@ma7payne
Copy link
Contributor Author

Mati, funciona de 10. Lo unico que habría que retocar es una restricción en RUP para evitar que el usuario pueda ..

  1. Iniciar prestaciones CON TURNO
  2. Iniciar prestaciones fuera de agenda
  3. Agregar sobreturnos del paciente restringido

Por ejemplo, para una restriccion hacia mi sobre el paciente matamala aldo en la imagen 1 no debería poder iniciar prestación ni ver huds, mientras que en la imagen 2 debería lanzarse un cartel invasivo advirtiendo que no tengo pemisos para iniciar prestaciones sobre ese paciente. Para el caso de la imagen 3 debebría funcionar igual que en la 2.

IMAGEN 1: image

IMAGEN 2: image

IMAGEN 3: image

@negro89 quedaron listos los cambios pedidos!

@ma7payne
Copy link
Contributor Author

TEST OK #11108

@ma7payne ma7payne added the test ok Los test estan ok label Jun 18, 2024
@negro89 negro89 removed the Aprobado label Jun 28, 2024
@ma7payne ma7payne force-pushed the HUDS-88 branch 2 times, most recently from cf925e1 to 7602965 Compare June 28, 2024 16:27
this.paciente = paciente;
this.resultadoBusqueda = [this.paciente];
this.darTurnoAutocitado();
if (!this.esPacienteRestringido(paciente)) {
Copy link
Contributor

@Fabio-Ramirez Fabio-Ramirez Jul 12, 2024

Choose a reason for hiding this comment

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

Se podría reemplazar la condición del if anterior por (paciente?.id)

@negro89 negro89 merged commit 3c971ae into master Jul 18, 2024
2 checks passed
@negro89 negro89 deleted the HUDS-88 branch July 18, 2024 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Aprobado test ok Los test estan ok
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants