Skip to content

MPI - Resolver el escaneo de nuevos DNIs - #3359

Open
Fabio-Ramirez wants to merge 1 commit into
masterfrom
MPI-472
Open

MPI - Resolver el escaneo de nuevos DNIs#3359
Fabio-Ramirez wants to merge 1 commit into
masterfrom
MPI-472

Conversation

@Fabio-Ramirez

Copy link
Copy Markdown
Contributor

Requerimiento

https://proyectos.andes.gob.ar/browse/MPI-472

Funcionalidad desarrollada

  1. Se agrega funcionalidad para contemplar los nuevos formatos de dni's
  2. En el archivo paciente-buscar.ts, se elimina linea de creacion de una interface sin uso.

Note

SUGERENCIA DE CASOS PARA REPRODUCIR:

  1. Probar con formato: NroTramite@Apellido@NombreCompleto@NroDni@Ejemplar@FechaNacimiento@FechaEmision@TokenValidacion.
  2. Ejemplo TokenValidacion: 2026.

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

@negro89 negro89 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Fabi, el pr está ok salvo por un detalle que no tuvimos en cuenta; en los nuevos DNI las fechas vienen con el año abreviado. Por lo tanto las fechas de nacimiento y emisión deberían ser del modo DD/MM/YY. Por ejemplo 23/01/89. Con esa modificación en el regex ya quedá de 10.

image

@negro89 negro89 added changes requested Se solicitaron cambios and removed Aprobado labels Jun 25, 2026
@Fabio-Ramirez

Copy link
Copy Markdown
Contributor Author

Fabi, el pr está ok salvo por un detalle que no tuvimos en cuenta; en los nuevos DNI las fechas vienen con el año abreviado. Por lo tanto las fechas de nacimiento y emisión deberían ser del modo DD/MM/YY. Por ejemplo 23/01/89. Con esa modificación en el regex ya quedá de 10.

image

Compa queda argegado este cambio.

@Fabio-Ramirez
Fabio-Ramirez requested a review from negro89 July 2, 2026 13:01
@Fabio-Ramirez Fabio-Ramirez added changes done and removed changes requested Se solicitaron cambios labels Jul 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants