-
Notifications
You must be signed in to change notification settings - Fork 5
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
base: master
Are you sure you want to change the base?
Conversation
Excelente revision mar! Ya subí las correciones. |
There was a problem hiding this 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:
-
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:
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:
-
Si es una cama en la que se cargó el préstamo, junto a su devolución, como se ve en sus movimientos:
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:
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.
There was a problem hiding this 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.
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. |
@MCele: @maring0019: Quedo a la espera del feedback 👍🏼 |
Revisión 3 - Mariela Videos: https://drive.google.com/drive/folders/1n2XUZyRAXv8ncKZYkXfeZnWAKjpuIUOC?usp=sharing |
Buenisimo campe, lo reviso. Gracias! |
Requerimiento
https://proyectos.andes.gob.ar/browse/IN-534
Funcionalidad desarrollada
UserStory llegó a completarse
Requiere actualizaciones en la base de datos
Requiere actualizaciones en la API
Requiere actualizaciones en andes-test-integracion