Skip to content

AM - Agregar control de correo electrónico al registrarse#245

Merged
negro89 merged 6 commits into
masterfrom
AM-366
Jun 18, 2026
Merged

AM - Agregar control de correo electrónico al registrarse#245
negro89 merged 6 commits into
masterfrom
AM-366

Conversation

@Fabio-Ramirez

@Fabio-Ramirez Fabio-Ramirez commented Apr 29, 2026

Copy link
Copy Markdown
Collaborator

Requerimiento

https://proyectos.andes.gob.ar/browse/AM-366

Funcionalidad desarrollada

  1. Se duplicó el ingreso del correo en la pantalla de registro, requiriendo que el paciente confirme el correo manualmente. Ambos campos deben coincidir para que el botón "Registrarme" se habilite.
  2. Controles agregados:
  • Campo "Confirmar Email": Se visualiza mientras y cuando el primer campo de mail este ok y el paciente debe tipear (obligatoriamente) nuevamente su correo y
  • Validación cruzada a nivel de formulario (emailsMatchValidator): compara ambos campos y bloquea el submit si no coinciden.
  • Mensaje de error: se muestra "Los e-mails ingresados no coinciden." cuando el campo fue tocado y los valores difieren.
  1. Comportamiento esperado
  • El paciente escanea su DNI e ingresa su celular.
  • Ingresa su correo en "Email" (permite pegar/autocompletar normalmente).
  • Ingresa nuevamente en "Confirmar Email" (debe tipear manualmente).
  • Si no coinciden → mensaje de error + botón deshabilitado.
  • Si coinciden y el formulario es válido → "Registrarme" habilitado.
  1. Archivo release.yml para github.

Note

SUGERENCIA DE CASOS PARA REPRODUCIR:

  1. Cuando el paciente no tiene usuario y tampoco profesional.
  2. Paciente con cuenta pero inactiva.
  3. Paciente con cuenta activa.
  4. Paciente sin cuenta pero si tiene como profesional.

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 requested review from a team as code owners April 29, 2026 15:35
@Fabio-Ramirez Fabio-Ramirez force-pushed the AM-366 branch 4 times, most recently from a66e356 to 46e22d1 Compare May 4, 2026 14:07
@Fabio-Ramirez Fabio-Ramirez force-pushed the AM-366 branch 2 times, most recently from 9f4b98f to 0b2f41a Compare May 13, 2026 13:08

@negro89 negro89 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Excelente funcionamiento ✅ Lo único que podriamos rever (de yapa) antes de mergear es el mensaje en la pantalla posterior al registro de un nuevo paciente que dice "si ya tenes tu codigo de activación tocá a continuación:". Como no damos una opción alternativa si el código no llegó, quizas lo mas apropieado seria algo como "Ahora ya podes loguearte por primera ves con tu usuario" (al botón tambien lo vi un poco mas chato que los otros)

@negro89 negro89 added the publicar Debería formar parte del próximo release label Jun 12, 2026

@negro89 negro89 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Aunque no esta ligado estríctamente a la tarea, si el usuario que se está registrando como paciente ya posee una cuenta de profesional, al escanear el documento matchea con con esta cuenta existente. Podríamos aprovechar de filtrar esta búsqueda para que solo compare con cuentas de perfil paciente.

@negro89 negro89 added the request changes cambios pendientes label Jun 16, 2026
@negro89 negro89 merged commit 596ccb2 into master Jun 18, 2026
2 checks passed
@negro89 negro89 deleted the AM-366 branch June 18, 2026 12:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

publicar Debería formar parte del próximo release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants