Skip to content

REC-190: Cambio de email con confirmación#118

Open
ma7payne wants to merge 1 commit intomasterfrom
REC-190
Open

REC-190: Cambio de email con confirmación#118
ma7payne wants to merge 1 commit intomasterfrom
REC-190

Conversation

@ma7payne
Copy link
Copy Markdown

@ma7payne ma7payne commented Jan 27, 2026

Requerimiento

https://proyectos.andes.gob.ar/browse/REC-190

Funcionalidad desarrollada

Note

Esta funcionalidad aplica a usuarios que poseen el rol de Farmacéutico

  1. Refactorización y Arquitectura

    • Movimiento de Componente: Se movió EditUserInfoComponent desde el módulo de profesionales a una ubicación compartida (src/app/shared/components/edit-user-info).
    • Creación de Módulo: Se creó EditUserInfoModule para encapsular este componente y permitir su importación tanto en el módulo de profesionales como en el de farmacéuticos.
  2. Enrutamiento (Routing)

    • Nuevas Rutas: Se agregó la ruta editar-usuario dentro de pharmacists-routing.module.ts.
    • Navegación Dinámica: Se actualizó el HeaderComponent para que el enlace "Editar información" redirija correctamente según el rol del usuario conectado:
      • Farmacias $\rightarrow$ /farmacias/editar-usuario
      • Profesionales $\rightarrow$ /profesionales/editar-usuario
  3. Lógica de Negocio Diferenciada:

    Se modificó EditUserInfoComponent para manejar lógica específica según el rol:

    • Rol Farmacia:
      • Al solicitar cambio de email, se dispara un flujo de confirmación por correo (el cambio no es inmediato).
      • Se actualiza tanto el email como el username (para mantener la consistencia en el login).
    • Rol Profesional: Mantiene el comportamiento original de actualización directa.
  4. Interfaz de Usuario (UI)

    • Página de Confirmación: Se crea la interfaz de ConfirmUpdateComponent para mostrar mensaje final luego de realizar los cambios en el email.

UserStory llegó a completarse

  • Si
  • No
  • No corresponde

Requiere actualizaciones en la base de datos

  • Si
  • No

Requiere actualizaciones en la API

@ma7payne ma7payne force-pushed the REC-190 branch 2 times, most recently from 2f82dfe to 653fa69 Compare February 2, 2026 14:31
@ma7payne ma7payne changed the title feat(REC): cambio de email con confirmación REC-190: Cambio de email con confirmación Mar 18, 2026
@ma7payne ma7payne force-pushed the REC-190 branch 2 times, most recently from c1b4669 to 2e1bf22 Compare April 29, 2026 13:32
Copy link
Copy Markdown

@agustin1996ra agustin1996ra left a comment

Choose a reason for hiding this comment

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

Falta quitar el cuadro de Obras sociales y el "ambito privado" del header

Despues el agente encontro una inconsitencia en como se comunican los mensajes de error:

Mensajería de error inconsistente en confirmación frontend. Backend responde con campo mensaje, pero frontend busca message.
Evidencia en

  • recetar-api/src/controllers/users.controller.ts#L102
  • recetar-app/src/app/auth/components/confirm-update/confirm-update.component.ts#L47.

@ma7payne
Copy link
Copy Markdown
Author

ma7payne commented May 7, 2026

Falta quitar el cuadro de Obras sociales y el "ambito privado" del header

Despues el agente encontro una inconsitencia en como se comunican los mensajes de error:

Mensajería de error inconsistente en confirmación frontend. Backend responde con campo mensaje, pero frontend busca message. Evidencia en

  • recetar-api/src/controllers/users.controller.ts#L102
  • recetar-app/src/app/auth/components/confirm-update/confirm-update.component.ts#L47.

Hechos los 3 cambios:

  1. header.component.html — Eliminada la línea <p>Ámbito privado</p> (líneas 13-14)
  2. pharmacists-form.component.html — Eliminado el card completo de "Obras sociales" ( líneas 79-111)
  3. confirm-update.component.ts:47 — Corregido err.error?.messageerr.error?.mensaje para alinearse con la respuesta del backend que usa el campo mensaje

@ma7payne ma7payne added the Changes done Cambios realizados label May 7, 2026
Copy link
Copy Markdown

@aldoEMatamala aldoEMatamala left a comment

Choose a reason for hiding this comment

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

funcionamiento ok!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1 Revisor Changes done Cambios realizados

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants