IN - Implementa periodos censables en internacion

[ma7payne]

Requerimiento

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

Funcionalidad desarrollada

1. En el detalle de la internación (Ingreso) puedo verse una opción para agregar periodos censables, solo en capa estadística.
2. El selector de periodos permite agregar uno o mas periodos a partir del ingreso por fechas.
3. Los periodos se agregan al campo periodosCensables de la internación.
4. Si se borra el listado de periodos el estado esCensable en la internación pasa a "false".
5. Si la internación tiene estado esCensable "false" y se agregan periodos, pasa a ser "true".

UserStory llegó a completarse

• Si
• No
• No corresponde

Requiere actualizaciones en la base de datos

• Si
• No

Requiere actualizaciones en la API

• Si IN - Implementa periodos censables en internacion api#1894
• No

Requiere actualizaciones en andes-test-integracion

• Si IN - Implementa periodos censables en internacion andes-test-integracion#520
• No

[ma7payne]

@AgosLizzi dejo el PR para revisar la interfaz, si esta ok el lugar donde se coloca el selector o cualquier sugerencia de diseño y comportamiento que creas apropiada. Quedo atento.

[AgosLizzi]

Mati, un par de observaciones:

1. O editamos o agregamos. SI tenemos un botón que dice "editar" no es necesario un tooltip, el botón ya indica.
   
2. Siguiendo en la misma linea que el item anterior. Yo modificaria la acción del botón según lo que tenemos. Si no tenemos nada cargado el botón que da la acción es un "AGREGAR". Y si realmente queremos "EDITAR" lo pondría sobre el listado que se va generando
   
3. Para el listado, agregaría un pleex table para diferenciar un periodo de otro. Decime que te parece
   
4. Esto resulta muy reiterativo.
   
5. Por último, Cuando elimino un concepto se abre un modal de advertencia?

[ma7payne]

@AgosLizzi los cambios charlados fueron realizados.

[negro89]

Mati, completando el circuito veo que lo datos se guardan correctamente y todo queda bien entendible desde el detalle en el sidebar. Pero noto algunas dificultades en el circuito de registro de los periodos que hacen a la funcionalidad un poco confusa:

1. Si completo las fechas de un nuevo periodo que deseo agregar y en lugar de clickear en el boton "check" presiono "guardar" se lanza un cartel de confirmación pero el periodo no queda guardado.
2. Similar al anterior. Si completo las fechas de un nuevo periodo y clickeo el "chheck" pero en lugar de guardar presiono la "X" no aparece el cartel pero tampoco se registra el periodo (Habiendo presionado previamente el boton "check").
3. Cada vez que agrego un periodo, este se almacena debajo del todo y se resetean los campos de fecha quedando listos para registrar otro periodo. Esta ok, pero en mi pantalla no se percibe a primera vista que se haya registrado el nuevo periodo, por lo que sugeriría agregar un scrollToEnd cada vez que se registra un periodo.

Como propuesta (Revisar con agos en todo caso) suprimiría el botón "guardar" asi por un lado no mezclamos texto e iconos en los botones, y por el otro eliminamos la ambigüedad de la acción de guardar. Y dejaría el botón check con un tooltip "guardar periodo" encargado de almacenar cada uno. Quizas con la propuesta deja medio descolocado al botón "editar", para lo que se me ocurren dos cosas; que se muestre solo si hay al menos un periodo censable o que cada periodo tenga un boton editar (De esta forma hasta se podría controlar que no se pisen los periodos)

Dejo videito con las observaciones hechas: https://www.loom.com/share/ae2c4734eea14ac385af1a922a3e457b?sid=b2eb58e5-e244-428f-96c3-f806e0e56bd4

---

P/D: Tres casos recurrentes al dia de hoy de fallo de test (Dos corridas distintas)

1. https://cloud.cypress.io/projects/xr7gft/runs/10068/test-results?actions=%5B%5D&browsers=%5B%5D&groups=%5B%5D&isFlaky=%5B%5D&modificationDateRange=%7B%22startDate%22%3A%221970-01-01%22%2C%22endDate%22%3A%222038-01-19%22%7D&orderBy=EXECUTION_ORDER&oses=%5B%5D&specs=%5B%5D&statuses=%5B%7B%22value%22%3A%22FAILED%22%2C%22label%22%3A%22FAILED%22%7D%5D&testingTypesEnum=%5B%5D
2. https://cloud.cypress.io/projects/xr7gft/runs/10067/test-results?actions=%5B%5D&browsers=%5B%5D&groups=%5B%5D&isFlaky=%5B%5D&modificationDateRange=%7B%22startDate%22%3A%221970-01-01%22%2C%22endDate%22%3A%222038-01-19%22%7D&orderBy=EXECUTION_ORDER&oses=%5B%5D&specs=%5B%5D&statuses=%5B%7B%22value%22%3A%22FAILED%22%2C%22label%22%3A%22FAILED%22%7D%5D&testingTypesEnum=%5B%5D

[ma7payne]

TEST OK #11090

[ma7payne]

Mati, completando el circuito veo que lo datos se guardan correctamente y todo queda bien entendible desde el detalle en el sidebar. Pero noto algunas dificultades en el circuito de registro de los periodos que hacen a la funcionalidad un poco confusa:

1. Si completo las fechas de un nuevo periodo que deseo agregar y en lugar de clickear en el boton "check" presiono "guardar" se lanza un cartel de confirmación pero el periodo no queda guardado.
2. Similar al anterior. Si completo las fechas de un nuevo periodo y clickeo el "chheck" pero en lugar de guardar presiono la "X" no aparece el cartel pero tampoco se registra el periodo (Habiendo presionado previamente el boton "check").
3. Cada vez que agrego un periodo, este se almacena debajo del todo y se resetean los campos de fecha quedando listos para registrar otro periodo. Esta ok, pero en mi pantalla no se percibe a primera vista que se haya registrado el nuevo periodo, por lo que sugeriría agregar un scrollToEnd cada vez que se registra un periodo.

Como propuesta (Revisar con agos en todo caso) suprimiría el botón "guardar" asi por un lado no mezclamos texto e iconos en los botones, y por el otro eliminamos la ambigüedad de la acción de guardar. Y dejaría el botón check con un tooltip "guardar periodo" encargado de almacenar cada uno. Quizas con la propuesta deja medio descolocado al botón "editar", para lo que se me ocurren dos cosas; que se muestre solo si hay al menos un periodo censable o que cada periodo tenga un boton editar (De esta forma hasta se podría controlar que no se pisen los periodos)

Dejo videito con las observaciones hechas: https://www.loom.com/share/ae2c4734eea14ac385af1a922a3e457b?sid=b2eb58e5-e244-428f-96c3-f806e0e56bd4

P/D: Tres casos recurrentes al dia de hoy de fallo de test (Dos corridas distintas)

1. https://cloud.cypress.io/projects/xr7gft/runs/10068/test-results?actions=%5B%5D&browsers=%5B%5D&groups=%5B%5D&isFlaky=%5B%5D&modificationDateRange=%7B%22startDate%22%3A%221970-01-01%22%2C%22endDate%22%3A%222038-01-19%22%7D&orderBy=EXECUTION_ORDER&oses=%5B%5D&specs=%5B%5D&statuses=%5B%7B%22value%22%3A%22FAILED%22%2C%22label%22%3A%22FAILED%22%7D%5D&testingTypesEnum=%5B%5D
2. https://cloud.cypress.io/projects/xr7gft/runs/10067/test-results?actions=%5B%5D&browsers=%5B%5D&groups=%5B%5D&isFlaky=%5B%5D&modificationDateRange=%7B%22startDate%22%3A%221970-01-01%22%2C%22endDate%22%3A%222038-01-19%22%7D&orderBy=EXECUTION_ORDER&oses=%5B%5D&specs=%5B%5D&statuses=%5B%7B%22value%22%3A%22FAILED%22%2C%22label%22%3A%22FAILED%22%7D%5D&testingTypesEnum=%5B%5D

@negro89 Listo los nuevos cambios para revisar. Se quita el botón de Guardar para aplicar cambios solo con el botón de check. No se aplica el tooltip en el check porque rompe la vista, se podría evaluar con @AgosLizzi una alternativa para este caso.

[MarianoCampetella]

Esta variable no se utiliza en ningún lado. Si se dejó por un posible uso en el futuro, entonces se puede dejar!

[ma7payne]

@MarianoCampetella Listo el cambio y las sugerencias sobre los subscribe deprecados!