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

IN - Prestamo/devolucion de cama #2851

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

IN - Prestamo/devolucion de cama #2851

wants to merge 2 commits into from

Conversation

negro89
Copy link
Contributor

@negro89 negro89 commented Apr 20, 2023

Requerimiento

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

Funcionalidad desarrollada

  1. Refactor de formulario de préstamo agregando un campo fecha de devolución para los prestamos en el pasado que puedan afectar movimientos posteriores ya existentes.

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

@negro89 negro89 requested review from a team as code owners April 20, 2023 12:55
@negro89 negro89 requested review from MCele and AgosLizzi April 20, 2023 12:55
@silviroa
Copy link
Contributor

USUARIO: lmlagos
BUILD NUMBER: 8731
CYPRESS RUN: 7906
TEST START: 2023-04-20T13:10:39.254Z
TOTAL: 361
SUCCESS: 351
FAIL: 8
SKIPPED: 2

@silviroa
Copy link
Contributor

USUARIO: lmlagos
BUILD NUMBER: 8733
CYPRESS RUN: 7908
TEST START: 2023-04-20T13:55:40.528Z
TOTAL: 378
SUCCESS: 371
FAIL: 5
SKIPPED: 2

@silviroa
Copy link
Contributor

USUARIO: lmlagos
BUILD NUMBER: 8742
CYPRESS RUN: 7917
TEST START: 2023-04-21T03:22:37.881Z
TOTAL: 177
SUCCESS: 132
FAIL: 45
SKIPPED: 0
⚠️⚠️⚠️⚠️
Hay archivos que no corrieron. Vuelva a intentar.

@silviroa
Copy link
Contributor

USUARIO: lmlagos
BUILD NUMBER: 8744
CYPRESS RUN: 7919
TEST START: 2023-04-21T13:24:47.584Z
TOTAL: 406
SUCCESS: 404
FAIL: 0
SKIPPED: 2

@silviroa silviroa added test ok Los test estan ok and removed test fail labels Apr 21, 2023
@maring0019 maring0019 self-requested a review May 11, 2023 17:21
@maring0019 maring0019 added the En revisión Se está probando label May 11, 2023
@negro89 negro89 added the prioritario Tarea con prioridad alta label May 15, 2023
maring0019

This comment was marked as resolved.

@maring0019 maring0019 added changes requested Se solicitaron cambios and removed En revisión Se está probando labels May 16, 2023
@maring0019 maring0019 self-requested a review May 16, 2023 18:19
maring0019

This comment was marked as resolved.

@negro89
Copy link
Contributor Author

negro89 commented Jun 1, 2023

Se adjunta un archivo con los errores encontrados. Cabe aclarar que las pruebas fueron realizadas en Capa Médica.

Errores-IN-534.pdf

Excelente revision mar! Ya subí las correciones.

@negro89 negro89 requested review from maring0019, silviroa and MarianoCampetella and removed request for AgosLizzi June 1, 2023 17:36
@negro89 negro89 added changes done and removed changes requested Se solicitaron cambios labels Jun 1, 2023
Copy link
Contributor

@MCele MCele left a comment

Choose a reason for hiding this comment

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

Observo errores al momento de editar una fecha de devolución de una cama.
Si estoy en el mapa en una fecha en el que la cama está prestada (dentro del periodo de préstamo-devolución), y luego se intenta editar la fecha/hora de la devolución:

  1. Si una cama ya tiene generada una fecha de devolución, de forma independiente, es decir se cargó su devolución después de generar el préstamo, como se ve en sus movimientos:
    image
    Si en el mapa de camas estamos dentro del periodo prestado, entonces si intentamos modificar la fecha/hora de la devolución se genera un nuevo movimiento de devolución de cama, en vez de modificar el original:
    image

  2. Si es una cama en la que se cargó el préstamo, junto a su devolución, como se ve en sus movimientos:
    image
    al intentar editar la fecha de devolución, se observa que se precarga la fecha/hora del mapa y no la de la devolución como debería, pero además al momento de editarlo ocurre un error no controlado en la api (patch /rup/internacion/camaEstados/:id) que responde con el mensaje:
    image

Por último, aunque no está definido en la tarea, al momento de cargar/editar fecha de devolución, como el periodo máximo (cuando existen ambas fechas) es de 15 días, debería aclararse esa información al usuario.

Copy link
Contributor

@maring0019 maring0019 left a comment

Choose a reason for hiding this comment

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

Se adjunta los errores que se ven.

Revision2.pdf

@maring0019 maring0019 added changes requested Se solicitaron cambios and removed changes done labels Sep 5, 2023
@maring0019
Copy link
Contributor

maring0019 commented Sep 5, 2023

Se adjunta un archivo con los errores encontrados. Cabe aclarar que las pruebas fueron realizadas en Capa Médica.
Errores-IN-534.pdf

Excelente revision mar! Ya subí las correciones.

Recién lo veo. No me llegó notificación de cambios. Hay algunas cosas que no funcionan bien y por ende no se pueden ver los últimos cambios hechos. En el mensaje de arriba subo el archivo con los errores que se ven.

@negro89
Copy link
Contributor Author

negro89 commented Sep 6, 2023

@MCele:
1 y 2: Recordemos que no se puede editar un préstamo, así como tampoco deshacerlo.
Se agregó leyenda que informa al usuario sobre el rango maximo de 15 dias entre fechas de préstamo/devolución.
image

@maring0019:
1: No pude ingresar un paciente en una cama ocupada, tendrás un video para seguirlo mejor?
2 y 3: Esta leyenda aparece cuando en otra capa la cama no está disponible. Es correcto ese funcionamiento 👍🏼 .
4: Yo los veo bien y al parecer cele tambien, tenes actualizado plex?
5: Los botones de acción no se habilitan hasta que todos los chequeos de fecha y recursos esten ok. Será por eso que no podes realizar un préstamo con conexión lenta?

Quedo a la espera del feedback 👍🏼

@maring0019
Copy link
Contributor

@MCele: 1 y 2: Recordemos que no se puede editar un préstamo, así como tampoco deshacerlo. Se agregó leyenda que informa al usuario sobre el rango maximo de 15 dias entre fechas de préstamo/devolución. image

@maring0019: 1: No pude ingresar un paciente en una cama ocupada, tendrás un video para seguirlo mejor? 2 y 3: Esta leyenda aparece cuando en otra capa la cama no está disponible. Es correcto ese funcionamiento 👍🏼 . 4: Yo los veo bien y al parecer cele tambien, tenes actualizado plex? 5: Los botones de acción no se habilitan hasta que todos los chequeos de fecha y recursos esten ok. Será por eso que no podes realizar un préstamo con conexión lenta?

Quedo a la espera del feedback 👍🏼

Revisión 3 - Mariela
Por un lado no pude reproducir esos errores. Salvo el punto 2 y 3, Esas camas no estaban ocupadas en las otras capas. Por otro lado adjunto archivo y videos con nueva revisión.

Revision3.pdf

Videos: https://drive.google.com/drive/folders/1n2XUZyRAXv8ncKZYkXfeZnWAKjpuIUOC?usp=sharing

@negro89 negro89 requested a review from MCele November 16, 2023 16:46
@negro89 negro89 added publicar and removed changes requested Se solicitaron cambios labels Nov 16, 2023
@MCele MCele removed the prioritario Tarea con prioridad alta label Dec 19, 2023
@MarianoCampetella
Copy link
Contributor

Noto un comportamiento raro a la hora de editar una cama una vez realizada la prestación/devolución de la misma!
SI bien la vista se observa normal, en la base de datos se me duplican los datos en la colección de estados!
image

Aguardo respuestas!

@MarianoCampetella MarianoCampetella added the changes requested Se solicitaron cambios label Jan 17, 2024
@negro89
Copy link
Contributor Author

negro89 commented Jan 26, 2024

Buenisimo campe, lo reviso. Gracias!

@negro89 negro89 removed the publicar label May 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changes requested Se solicitaron cambios test ok Los test estan ok
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants