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

IN - Duplicación de egresos #2980

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

IN - Duplicación de egresos #2980

wants to merge 1 commit into from

Conversation

MarianoCampetella
Copy link
Contributor

@MarianoCampetella MarianoCampetella commented Feb 22, 2024 •
edited
Loading

Requerimiento

https://proyectos.andes.gob.ar/browse/IN-593

Funcionalidad desarrollada

  1. Se corrige bug de que al momento de editar un egreso en capa estadística, se duplicaban los mismos!
  2. Se corrige bug de que al momento de egresar un paciente en el pasando cuando en el presente se hace lo mismo, no se generen egresos duplicados con diferentes fechas.

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

@MarianoCampetella MarianoCampetella changed the title fix(IN-593): Duplicacion de egresos IN - Duplicación de egresos Feb 22, 2024
@silviroa
Copy link
Contributor

USUARIO: mcampetella
BUILD NUMBER: 10528
CYPRESS RUN: 9629
TEST START: 2024-02-22T13:10:48.074Z
TOTAL: 399
SUCCESS: 389
FAIL: 8
SKIPPED: 2

@silviroa
Copy link
Contributor

USUARIO: mcampetella
BUILD NUMBER: 10529
CYPRESS RUN: 9630
TEST START: 2024-02-22T13:25:46.154Z
TOTAL: 371
SUCCESS: 359
FAIL: 10
SKIPPED: 2

@silviroa
Copy link
Contributor

USUARIO: mcampetella
BUILD NUMBER: 10558
CYPRESS RUN: 9652
TEST START: 2024-02-26T15:50:46.342Z
TOTAL: 399
SUCCESS: 396
FAIL: 1
SKIPPED: 2

@silviroa
Copy link
Contributor

USUARIO: mcampetella
BUILD NUMBER: 10559
CYPRESS RUN: 9653
TEST START: 2024-02-26T17:19:54.647Z
TOTAL: 399
SUCCESS: 397
FAIL: 0
SKIPPED: 2

@silviroa silviroa added test ok Los test estan ok and removed test fail labels Feb 26, 2024
Fabio-Ramirez
Fabio-Ramirez requested changes Apr 22, 2024
View reviewed changes
Copy link
Contributor

@Fabio-Ramirez Fabio-Ramirez left a comment

Choose a reason for hiding this comment

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

Campe, el código lo veo bien. Lo único que recalco es lo que estuvimos viendo que la opcion "desocupar" se sigue visualizando. Adjunto imagenes. Este seria el caso del paciente egresado el 18/03/2024
image y cuando lo "busco" aparece la opción "descocupar"
image

@negro89 negro89 added the En pausa Temporalmente pausado label May 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Fabio-Ramirez Fabio-Ramirez Fabio-Ramirez requested changes

@ma7payne ma7payne Awaiting requested review from ma7payne ma7payne is a code owner automatically assigned from andes/code-review

@AgosLizzi AgosLizzi Awaiting requested review from AgosLizzi AgosLizzi is a code owner automatically assigned from andes/code-review

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
En pausa Temporalmente pausado test ok Los test estan ok
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@MarianoCampetella @silviroa @Fabio-Ramirez @negro89