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 (Fix) - Duplicación de egresos y otros #3004

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

Conversation

negro89
Copy link
Contributor

@negro89 negro89 commented Apr 26, 2024

Requerimiento

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

Funcionalidad desarrollada

  1. Evita duplicación de egresos al editar desde el listado
  2. Oculta opciones para "desocupar cama" en el dropdown del mapa (al igual que en el sidebar) si el paciente está egresado en el futuro
  3. Al editar una cama, restringe edición de UO si la cama se encuentra ocupada en ALGUNA capa
  4. Oculta tab "egreso" en todas las capas desde el mapa. Las habilita solo desde el listado y el historial

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

@negro89
Copy link
Contributor Author

negro89 commented May 6, 2024

tests al 06/05/2024
image
image

@negro89 negro89 added the test ok Los test estan ok label May 6, 2024
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.

Lauchis en general lo veo bien, porque ya no se duplican los egresos, agregaría dos controles:

  • una vez que el paciente fue egresado, no debería visualizarse el botón "desocupar cama" con sus opciones (aunque se haya quitado la internacion). Ej:
    image

  • Al intentar modificar un egreso en la internación (desde historial), se puede colocar cualquier fecha de egreso, incluso una por anterior al ultimo movimiento, y cuando se la carga aparece un mensaje de error. Si bien la internación se modifica en la fecha de egreso, no se refleja el cambio en el movimiento. Ej del mensaje:
    image
    Resultado de las fechas:
    image
    En las fechas se puede observar que no corresponde la fecha de egreso de la internacion, con la del movimiento, y además la fecha (concretamente hora) es menor a la que tiene el movimiento anterior.

Por otro lado, veo que:

1-no se visualiza en capa estadística los datos del egreso:

image

2-cuando se completa la fecha de egreso con un formato inválido de fecha, tira un error en la consola:

image

@negro89
Copy link
Contributor Author

negro89 commented Jul 5, 2024

Lauchis en general lo veo bien, porque ya no se duplican los egresos, agregaría dos controles:

  • una vez que el paciente fue egresado, no debería visualizarse el botón "desocupar cama" con sus opciones (aunque se haya quitado la inetrnacion).
  • Al intentar modificar un egreso en la internación (desde historial), se puede colocar cualquier fecha de egreso, incluso una por anterior al ultimo movimiento, y cuando se la carga aparece un mensaje de error. Si bien la internación se modifica en la fecha de egreso, no se refleja el cambio en el movimiento. Ej del mensaje:

En las fechas se puede observar que no corresponde la fecha de egreso de la internacion, con la del movimiento, y además la fecha (concretamente hora) es menor a la que tiene el movimiento anterior.

Por otro lado, veo que:

1-no se visualiza en capa estadística los datos del egreso:

2-cuando se completa la fecha de egreso con un formato inválido de fecha, tira un error en la consola:

Todos los cambios hechos, espero nueva revision 👌🏼

@@ -108,17 +111,19 @@ export class InternacionDetalleComponent implements OnInit, AfterViewChecked {
// Configura los tabs a mostrar según capa y vista
this.mapaCamasService.resumenInternacion$.pipe(
map(resumen => {
if (!!this.editar && this.editar !== resumen.paciente?.id) {
if (!!this.editarIngreso && this.editarIngreso !== resumen.paciente?.id) {
Copy link
Contributor

Choose a reason for hiding this comment

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

Aca se podria evaluar la condición !!this.editarIngreso por this.editarIngreso. Además si el tipo de editarIngreso es boolena, condice compararlo con un tipo de dato distinto como lo es un id?

@MCele MCele self-requested a review July 17, 2024 15:42
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.

Lauchita, con respecto a los cambios que te comenté anteriormente, vi que se todos, excepto uno que es el la primer observación que te dejé, respecto a que desde capa estadítica, al editar un egreso desde el mapa de camas, 1-no se visualiza los datos (sidebar vacío).
Te dejo los controles que me parece hay que mejorar por capas:

  1. Capa estádistica: todo bien, excepto lo antes mencionado (igual ahi queda evaluar si seguimos mostrando desde el mapa de camas ese tab).

  2. Capa médica:

  • Cuando se edita el egreso desde el historial se duplica el movimiento, y no sólo se genera un nuevo movimiento de egreso, si no que si hubo en el medio un cambio de cama, se genera sobre otra cama ese nuevo egreso. Ejemplo:
    image
  • Cuando se genera un nuevo egreso pareciera como si quedara algún observable o acción sin cerrar. Ej: https://app.screencastify.com/v3/watch/cd0wr7Fb20cZeTqMpJv6

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changes done test ok Los test estan ok
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants