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

INT - Fix edición de egreso en internación validada #2269

Merged
merged 1 commit into from
May 27, 2021
Merged

Conversation

negro89
Copy link
Contributor

@negro89 negro89 commented May 12, 2021

Requerimiento

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

Funcionalidad desarrollada

  1. Agrega propiedad readonly en campos de egreso cuando la internación ya fue validada

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 requested review from a team as code owners May 12, 2021 15:37
@liquid36
Copy link
Contributor

USUARIO: lmlagos
BUILD NUMBER: 4033
CYPRESS RUN: 3071
TEST START: 2021-05-12T16:11:11.404Z
TOTAL: 385
SUCCESS: 377
FAIL: 8
SKIPPED: 0

@liquid36
Copy link
Contributor

Recordar que habiamos hablado de mostrar la componente cuando se puede editar. En la US se la linea de código donde se usa.

@palita1991 palita1991 self-requested a review May 14, 2021 18:29
Copy link
Contributor

@juuliotero juuliotero left a comment

Choose a reason for hiding this comment

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

Me pasa algo raro con los datos, como que tardan en cargar. Pero fuera de eso, cuando guardo cambios para un egreso, me deshabilita el tilde y si cambio de internación me lo deja deshabilitado. Sumo video.

https://www.loom.com/share/2faed09ae3fe435eb304bb0c027ffc0d

@negro89 negro89 force-pushed the IN-269 branch 2 times, most recently from e07f279 to ca27aaf Compare May 14, 2021 20:14
@negro89
Copy link
Contributor Author

negro89 commented May 14, 2021

@juuliotero si, probaste el pr justo cuando estaba haciendo cambios. De todas formas hay una query que está demorando mucho, habría que investigar bien.

@liquid36 ya estan corregidos los cambios.

@negro89 negro89 requested a review from juuliotero May 14, 2021 20:36
@liquid36
Copy link
Contributor

USUARIO: mbotta
BUILD NUMBER: 4074
CYPRESS RUN: 3110
TEST START: 2021-05-17T12:49:17.726Z
TOTAL: 306
SUCCESS: 279
FAIL: 27
SKIPPED: 0
⚠️⚠️⚠️⚠️
Hay archivos que no corrieron. Vuelva a intentar.

@liquid36
Copy link
Contributor

USUARIO: lmlagos
BUILD NUMBER: 4076
CYPRESS RUN: 3112
TEST START: 2021-05-17T13:19:07.241Z
TOTAL: 313
SUCCESS: 294
FAIL: 19
SKIPPED: 0
⚠️⚠️⚠️⚠️
Hay archivos que no corrieron. Vuelva a intentar.

@liquid36
Copy link
Contributor

USUARIO: mbotta
BUILD NUMBER: 4079
CYPRESS RUN: 3115
TEST START: 2021-05-17T13:54:38.344Z
TOTAL: 388
SUCCESS: 375
FAIL: 13
SKIPPED: 0

@liquid36
Copy link
Contributor

Revisar que fallan unos test de internación

@liquid36
Copy link
Contributor

USUARIO: mbotta
BUILD NUMBER: 4148
CYPRESS RUN: 3183
TEST START: 2021-05-21T17:01:35.762Z
TOTAL: 291
SUCCESS: 275
FAIL: 16
SKIPPED: 0
⚠️⚠️⚠️⚠️
Hay archivos que no corrieron. Vuelva a intentar.

@liquid36
Copy link
Contributor

USUARIO: mbotta
BUILD NUMBER: 4156
CYPRESS RUN: 3190
TEST START: 2021-05-21T17:54:00.481Z
TOTAL: 393
SUCCESS: 393
FAIL: 0
SKIPPED: 0

@liquid36 liquid36 added test ok Los test estan ok and removed test fail labels May 21, 2021
@liquid36 liquid36 merged commit 9c031c8 into master May 27, 2021
@liquid36 liquid36 deleted the IN-269 branch May 27, 2021 16:38
liquid36 pushed a commit that referenced this pull request Jun 2, 2021
# [5.49.0](v5.48.0...v5.49.0) (2021-06-02)

### Bug Fixes

* **buscador-ficha:** doble request ([46df6a4](46df6a4))
* **COM:** bug buscar org origen ([56ae829](56ae829))
* **ficha:** clasificacion y loader buscar paciente ([5aab87c](5aab87c))
* **mapa-camas:** edición de egreso en internación validada ([#2269](#2269)) ([9c031c8](9c031c8))
* **mpi:** quita regex del nombre ([#2298](#2298)) ([2fb85b1](2fb85b1))
* **rup:** cambia solicitante por profesional ([#2294](#2294)) ([88da8e6](88da8e6))
* **top:** deshabilita boton hasta no cargar datos requeridos en formulario ([#2288](#2288)) ([410056d](410056d))

### Features

* **ficha-epidemiologica:** mensaje no hay fichas ([b247443](b247443))
* **fichaEpidemiologia:** Se agrega al pacientes la información de identificación si es extranjero ([48e0dc9](48e0dc9))
* **mapa-camas:** columna obra social en listado internación ([#2290](#2290)) ([f1df855](f1df855))
* **mapa-camas:** muestra la prestacion de estadistica en resumen de internacion ([#2299](#2299)) ([f0029ae](f0029ae))
* **mapa-camas:** obra social prepaga ([#2285](#2285)) ([0b7fd1b](0b7fd1b))
* **mpi:**  se agrega idPacientePrincipal en  paciente ([#2295](#2295)) ([5c3b228](5c3b228))
* **rup:** agregar unidad organizativa a la prestación ([#2300](#2300)) ([398884e](398884e))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Aprobado test ok Los test estan ok
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants