Skip to content

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

Open
Fabio-Ramirez wants to merge 5 commits into
masterfrom
AM-366
Open

AM - Agregar control de correo electrónico al registrarse#245
Fabio-Ramirez wants to merge 5 commits into
masterfrom
AM-366

Conversation

@Fabio-Ramirez
Copy link
Copy Markdown
Collaborator

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

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

  • Si
  • No

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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants