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.

Sign up for GitHub

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

Jump to bottom

GDU - Controlar datos del profesional asociado al usuario #3018

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

GDU - Controlar datos del profesional asociado al usuario #3018

wants to merge 1 commit into from

Conversation

MarianoCampetella
Copy link
Contributor

@MarianoCampetella MarianoCampetella commented Jun 3, 2024 •
edited
Loading

Requerimiento

https://proyectos.andes.gob.ar/browse/GDU-64

Funcionalidad desarrollada

  1. Se verifica si el usuario es un profesional habilitado o no. Si se encuentra duplicado mostrará los datos del habilitado y en caso de que se encuentre deshabilitado mostrará un badge rojo DESHABILITADO.

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

@MarianoCampetella MarianoCampetella requested review from a team as code owners June 3, 2024 12:53
@MarianoCampetella MarianoCampetella added the test ok Los test estan ok label Jun 3, 2024
@aldoEMatamala
Copy link
Contributor

campe como lo hablamos te dejo por escrito las observaciones:
1- Si es correcta la interpretacion de cuando un profesional se encuentre deshabilitado se puedan dar permisos desde el gestor.
2- En la visualización de "MIS DATOS" se ven las profesiones de grado de otro profesional
imagen

@aldoEMatamala aldoEMatamala added the changes requested Se solicitaron cambios label Jun 4, 2024
@MarianoCampetella
Copy link
Contributor Author

MarianoCampetella commented Jun 5, 2024 •
edited
Loading

campe como lo hablamos te dejo por escrito las observaciones: 1- Si es correcta la interpretacion de cuando un profesional se encuentre deshabilitado se puedan dar permisos desde el gestor. 2- En la visualización de "MIS DATOS" se ven las profesiones de grado de otro profesional imagen

Corregido, ahora cada vez que se vaya a loguear un usuario siendo el mismo un profesional, el cual este duplicado por BD, va a seleccionar el que se encuentre habilitado. En caso de que se encuentren ambos deshabilitados selecciona uno de ellos.

@MarianoCampetella MarianoCampetella added changes done and removed changes requested Se solicitaron cambios labels Jun 5, 2024
@aldoEMatamala
Copy link
Contributor

campe como lo hablamos te dejo por escrito las observaciones: 1- Si es correcta la interpretacion de cuando un profesional se encuentre deshabilitado se puedan dar permisos desde el gestor. 2- En la visualización de "MIS DATOS" se ven las profesiones de grado de otro profesional imagen

Corregido, ahora cada vez que se vaya a loguear un usuario siendo el mismo un profesional, el cual este duplicado por BD, va a seleccionar el que se encuentre habilitado. En caso de que se encuentren ambos deshabilitados selecciona uno de ellos.

genial campe ahi lo probe y anda joya

negro89
negro89 requested changes Jun 11, 2024
View reviewed changes
src/app/app.component.ts Outdated
if (profesional) {
this.profesional = profesional;
this.auth.setProfesionalHabilitado(profesional.habilitado);
this.profesionalService.get({ documento: this.auth.usuario.documento }).subscribe(profesional => {
Copy link
Contributor

Choose a reason for hiding this comment

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

Campe hay un problema con esta funcion; como está hecha para una búsqueda tipo filtro, si el dni que se está buscando es subcadena de otro dni entonces también lo va a traer. Habría que ..

  1. modificar la búsqueda para que ofrezca la opcion de matching exacto
  2. controlar en el response que efectivamente se trate de la misma persona

(En ambos casos aclararía la situación en el codigo con algun comentario)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Corregido, se agregó un parametro mas llamado busquedaExacta que se utiliza desde la API para hacer un matcheo exacto con el profesional.

src/app/app.component.ts
Comment on lines +103 to +110
const profHabilitado = profesional.find(prof => prof.habilitado);
this.profesional = profHabilitado || profesional[0];
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
const profHabilitado = profesional.find(prof => prof.habilitado);
this.profesional = profHabilitado || profesional[0];
this.profesional = profesional.filter(prof => prof.habilitado);

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Este comentario se ignora ya que se aplica la primera sugerencia de cambio.

src/app/apps/gestor-usuarios/components/usuario-detalle/usuario-detalle.component.ts

this.profesional = profesional[0];
this.profesionalHabilitado = profesional.find(prof => prof.habilitado === true);
this.profesional = this.profesionalHabilitado || profesional[0];
Copy link
Contributor

Choose a reason for hiding this comment

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

idem a la sugerencia anterior

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Idem al comentario anterior!

@negro89 negro89 added changes requested Se solicitaron cambios and removed changes done labels Jun 11, 2024
@MarianoCampetella MarianoCampetella added changes done and removed changes requested Se solicitaron cambios test ok Los test estan ok labels Jun 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ma7payne ma7payne ma7payne approved these changes

@Fabio-Ramirez Fabio-Ramirez Fabio-Ramirez approved these changes

@aldoEMatamala aldoEMatamala aldoEMatamala approved these changes

@negro89 negro89 Awaiting requested review from negro89

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
Aprobado changes done
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@MarianoCampetella @aldoEMatamala @ma7payne @negro89 @Fabio-Ramirez