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.

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

feat(MPI): No mostrar inactivos en las relaciones #2662

Merged
merged 4 commits into from
Jun 16, 2022
Merged

feat(MPI): No mostrar inactivos en las relaciones #2662

merged 4 commits into from
Jun 16, 2022

Conversation

leon-bravo
Copy link
Contributor

Requerimiento

MPI-383 - No mostrar pacientes inactivos en las relaciones de un paciente

Funcionalidad desarrollada

  1. Se modifico el componente de visualización de las relaciones de pacientes para que solo se incluyan los que estan activos

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

@leon-bravo leon-bravo requested review from a team as code owners May 26, 2022 16:53
@martinebucarey
Copy link
Contributor

USUARIO: lbravo
BUILD NUMBER: 6509
CYPRESS RUN: 5390
TEST START: 2022-05-26T16:59:55.659Z
TOTAL: 406
SUCCESS: 405
FAIL: 0
SKIPPED: 1

@martinebucarey martinebucarey added the test ok Los test estan ok label May 26, 2022
Copy link
Contributor

@palita1991 palita1991 left a comment

Choose a reason for hiding this comment

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

https://www.loom.com/share/446df8b870ad4e3eb8a36d0bfaef0c6c
La funcionalidad de inactivar paciente y que luego no aparezcan en el listado, se ve que funciona bien. Pero me dio duda que al principio antes de inactivar al paciente el listado ya era distinto, en mi local estaba conectado a la BD de demo para comparar.

@MCele
Copy link
Contributor

MCele commented May 31, 2022

imagen
me tira un error en la consola con algunos pacientes. Creo que es en aquellos que no contienene el campo inactivo. (estimo que es lo que muestra el Pala en el video, ya que el paciente que no coincide no tiene el campo "inactivo"). Se debería agregar un control para estos casos

@leon-bravo
Copy link
Contributor Author

https://www.loom.com/share/446df8b870ad4e3eb8a36d0bfaef0c6c La funcionalidad de inactivar paciente y que luego no aparezcan en el listado, se ve que funciona bien. Pero me dio duda que al principio antes de inactivar al paciente el listado ya era distinto, en mi local estaba conectado a la BD de demo para comparar.

Gracias palita, ahi pude resolverlo, gracias a tu comentario y al de cele. Creo que ya quedo

@leon-bravo
Copy link
Contributor Author

imagen me tira un error en la consola con algunos pacientes. Creo que es en aquellos que no contienene el campo inactivo. (estimo que es lo que muestra el Pala en el video, ya que el paciente que no coincide no tiene el campo "inactivo"). Se debería agregar un control para estos casos

Cele tenias razón, el problema se daba justamente por aquellos pacientes que no tienen el campo activo, el mismo problema ocurre aparentemente en la funcionalidad de inactivar. En el listado de relaciones

@MCele MCele merged commit aa41a6d into master Jun 16, 2022
@MCele MCele deleted the MPI-383 branch June 16, 2022 18:33
liquid36 pushed a commit that referenced this pull request Jun 22, 2022
# [5.99.0](v5.98.0...v5.99.0) (2022-06-22)

### Features

* **IN:** motivo suspension indicacion ([#2665](#2665)) ([ef5c3fe](ef5c3fe))
* **int:** mejoras visales plan ind ([#2676](#2676)) ([396eb56](396eb56))
* **INT:** plan ind mejoras ejecucion ([#2677](#2677)) ([4972e90](4972e90))
* **MPI:** No mostrar inactivos en las relaciones ([#2662](#2662)) ([aa41a6d](aa41a6d))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test ok Los test estan ok
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants