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

MPI: Implementa registro para extranjeros con DNI argentino #2822

Merged
merged 1 commit into from
Dec 15, 2023
Merged

MPI: Implementa registro para extranjeros con DNI argentino #2822

merged 1 commit into from
Dec 15, 2023

Conversation

ma7payne
Copy link
Contributor

@ma7payne ma7payne commented Feb 27, 2023 •
edited by Fabio-Ramirez
Loading

Requerimiento

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

Funcionalidad desarrollada

  1. Se crea un nuevo formulario (paralelo) para registrar a un paciente extranjero con DNI argentino
  2. Se crea una opción (boton slide) para habilitar el con DNI a un paciente extranjero
  3. Se validan los campos actuales del formulario para el paciente extranjero al momento de habilitar la opción para registrar con DNI argentino
  4. Se validan los campos del paciente con DNI argentino
  5. Al momento de validar con RENAPER, se reemplazan los datos obtenidos en el formulario de paciente con DNI argentino
  6. Al momento de guardar los datos del paciente, se chequean las suregencias con pacientes similares
  7. Al seleccionar un paciente sugerido, el formulario se redirige al formulario tradicional de registro de paciente argentino
  8. Al momento de redirigir, se muestra una alerta informando acerca de la vinculación del paciente extranjero con el nuevo paciente con DNI argentino
  9. Al guardar los datos (con o sin paciente sugerido) se actualizan en DB los datos del paciente extranjero y se crea el nuevo paciente con DNI argentino
  10. Se vincula al paciente extranjero actual con el nuevo paciente con DNI argentino.
  11. En base se guarda en el atributo "identificadores" el vinculo del extranjero.

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

@ma7payne ma7payne requested review from a team as code owners February 27, 2023 16:20
@silviroa
Copy link
Contributor

silviroa commented Mar 2, 2023

USUARIO: mchiappe
BUILD NUMBER: 8514
CYPRESS RUN: 7693
TEST START: 2023-03-02T18:18:02.112Z
TOTAL: 347
SUCCESS: 341
FAIL: 4
SKIPPED: 2

AgosLizzi
AgosLizzi requested changes Mar 3, 2023
View reviewed changes
Copy link
Contributor

@AgosLizzi AgosLizzi left a comment

Choose a reason for hiding this comment

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

Mati, detallitos:

  1. Primera vez que veo un botón "seleccionar" en ANDES. No me parece muy usable para lo que estamos acostumbrados en el sistema. La acción puede salir directamente del click sobre el listado, como se suelen comportar el resto de las acciones en la interfaz.
    2023-03-03 (4)

  2. Ampliaria un poco el tamaño del sidebar para que el listado se vea asi:
    2023-03-03 (5)

@silviroa
Copy link
Contributor

USUARIO: mchiappe
BUILD NUMBER: 8575
CYPRESS RUN: 7753
TEST START: 2023-03-13T17:41:39.259Z
TOTAL: 406
SUCCESS: 394
FAIL: 10
SKIPPED: 2

@silviroa
Copy link
Contributor

USUARIO: mchiappe
BUILD NUMBER: 8641
CYPRESS RUN: 7817
TEST START: 2023-03-28T14:09:12.150Z
TOTAL: 406
SUCCESS: 400
FAIL: 4
SKIPPED: 2

@ma7payne ma7payne force-pushed the MPI-393 branch 2 times, most recently from 04ca64a to cc37069 Compare April 20, 2023 15:12
@silviroa
Copy link
Contributor

USUARIO: mchiappe
BUILD NUMBER: 8739
CYPRESS RUN: 7914
TEST START: 2023-04-20T15:42:03.495Z
TOTAL: 356
SUCCESS: 350
FAIL: 4
SKIPPED: 2

@ma7payne ma7payne force-pushed the MPI-393 branch 2 times, most recently from 317908d to 032a351 Compare April 21, 2023 13:27
@silviroa
Copy link
Contributor

USUARIO: mchiappe
BUILD NUMBER: 8746
CYPRESS RUN: 7921
TEST START: 2023-04-21T13:55:21.576Z
TOTAL: 356
SUCCESS: 350
FAIL: 4
SKIPPED: 2

@MarianoCampetella
Copy link
Contributor

Es necesario modificar los siguientes test
image

@MarianoCampetella MarianoCampetella added the changes requested Se solicitaron cambios label May 8, 2023
@negro89 negro89 added the prioritario Tarea con prioridad alta label May 15, 2023
@silviroa
Copy link
Contributor

USUARIO: cramos
BUILD NUMBER: 9989
CYPRESS RUN: 9098
TEST START: 2023-11-28T16:31:08.556Z
TOTAL: 325
SUCCESS: 313
FAIL: 10
SKIPPED: 2
⚠️⚠️⚠️⚠️
Hay archivos que no corrieron. Vuelva a intentar.

@silviroa
Copy link
Contributor

USUARIO: mchiappe
BUILD NUMBER: 9991
CYPRESS RUN: 9100
TEST START: 2023-11-28T19:08:32.936Z
TOTAL: 363
SUCCESS: 328
FAIL: 33
SKIPPED: 2

@silviroa
Copy link
Contributor

USUARIO: mchiappe
BUILD NUMBER: 9996
CYPRESS RUN: 9105
TEST START: 2023-11-29T12:07:52.104Z
TOTAL: 319
SUCCESS: 282
FAIL: 35
SKIPPED: 2
⚠️⚠️⚠️⚠️
Hay archivos que no corrieron. Vuelva a intentar.

@silviroa
Copy link
Contributor

USUARIO: cramos
BUILD NUMBER: 9999
CYPRESS RUN: 9108
TEST START: 2023-11-29T13:15:39.784Z
TOTAL: 380
SUCCESS: 369
FAIL: 9
SKIPPED: 2

@MCele
Copy link
Contributor

MCele commented Dec 5, 2023

Revisar test

@MCele MCele added the muy prioritario Tarea con prioridad muy alta label Dec 5, 2023
@silviroa
Copy link
Contributor

USUARIO: lmlagos
BUILD NUMBER: 10060
CYPRESS RUN: 9168
TEST START: 2023-12-15T14:29:00.676Z
TOTAL: 372
SUCCESS: 361
FAIL: 9
SKIPPED: 2

@silviroa
Copy link
Contributor

USUARIO: lmlagos
BUILD NUMBER: 10061
CYPRESS RUN: 9169
TEST START: 2023-12-15T14:49:08.994Z
TOTAL: 406
SUCCESS: 404
FAIL: 0
SKIPPED: 2

@silviroa silviroa added test ok Los test estan ok and removed test fail labels Dec 15, 2023
@negro89 negro89 added Aprobado and removed prioritario Tarea con prioridad alta changes done labels Dec 15, 2023
@negro89 negro89 merged commit 1e9e090 into master Dec 15, 2023
2 checks passed
@negro89 negro89 deleted the MPI-393 branch December 15, 2023 16:06
liquid36 pushed a commit that referenced this pull request Dec 20, 2023
@semantic-release-bot
chore(release): 5.129.0 [skip ci]
5fee9ca 
# [5.129.0](v5.128.2...v5.129.0) (2023-12-20)

### Features

* **CIT:** reasigna turnos en agenda suspendida desde TOP ([#2927](#2927)) ([9ffe7b8](9ffe7b8))
* **MPI:** implementa form para extranjeros con DNI ([#2822](#2822)) ([1e9e090](1e9e090))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AgosLizzi AgosLizzi AgosLizzi approved these changes

@MarianoCampetella MarianoCampetella MarianoCampetella approved these changes

@negro89 negro89 negro89 approved these changes

@MCele MCele MCele approved these changes

Assignees
No one assigned
Labels
Aprobado muy prioritario Tarea con prioridad muy alta test ok Los test estan ok
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@ma7payne @silviroa @MarianoCampetella @MCele @negro89 @AgosLizzi