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

CIT - Implementa mockups de demanda insatisfecha #3006

Merged
merged 1 commit into from
Aug 2, 2024
Merged

Conversation

ma7payne
Copy link
Contributor

@ma7payne ma7payne commented May 13, 2024

Requerimiento

https://proyectos.andes.gob.ar/browse/CIT-277
https://proyectos.andes.gob.ar/browse/CIT-278

Funcionalidad desarrollada

  1. Implementa mockups para nuevo módulo de demanda insatisfecha
  2. Se recupera lista de espera de pacientes sin turnos, con opciones de filtros
  3. Se visualiza detalle de paciente en sidebar
  4. Se visualizan llamados con formulario de carga
  5. Se visualiza historial de turnos del paciente

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

@ma7payne ma7payne requested review from a team as code owners May 13, 2024 13:28
@ma7payne ma7payne requested review from MCele and negro89 May 13, 2024 13:28
@ma7payne ma7payne force-pushed the CIT-277 branch 4 times, most recently from d6da9f8 to 701f7b6 Compare May 13, 2024 13:56
@ma7payne
Copy link
Contributor Author

Test OK: #10875

@ma7payne ma7payne added the test ok Los test estan ok label May 13, 2024
@ma7payne ma7payne force-pushed the CIT-277 branch 3 times, most recently from 55b9280 to bb46a7a Compare May 15, 2024 17:47
@negro89 negro89 added publicar and removed publicar labels May 17, 2024
@ma7payne ma7payne added the En pausa Temporalmente pausado label May 20, 2024
@ma7payne ma7payne force-pushed the CIT-277 branch 2 times, most recently from e88f4d8 to 39e0a0a Compare May 23, 2024 15:11
@ma7payne ma7payne removed the En pausa Temporalmente pausado label May 24, 2024
@ma7payne ma7payne force-pushed the CIT-277 branch 2 times, most recently from 52e815b to 3735493 Compare May 24, 2024 13:34
@ma7payne
Copy link
Contributor Author

Test OK: #10942

@ma7payne ma7payne force-pushed the CIT-277 branch 3 times, most recently from 73eb7e2 to ac46084 Compare May 30, 2024 14:44
Copy link
Contributor

@AgosLizzi AgosLizzi left a comment

Choose a reason for hiding this comment

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

Mati, te dejo unas observaciones:

  • Como cambio importante: El filtro por prestación, por motivo y por paciente parecen no tener funcionalidad, es decir, sobre el listado no me aparece exclusivamente lo que seleccioné para filtrar. Dejo ejemplo
    filtro

  • Sobre el listado se muestra el estado "resuelto", no debería aparecer.
    resuelto

  • Parece que los llamados no se guardan. Una vez ingresado uno, doy volver y vuelvo a elegir mismo paciente, y no me muestra nada. Dejo video
    https://www.loom.com/share/26461ca215c3466283c86d6c207c436a

Como cambios secundarios:

  • Sobre el plex item se podría subir el tamaño de la tipo? Siento que el size:S va a quedar muy chico. Que te parece?
    tituosidebar
  • Al listado de pacientes, se le puede aplicar el mouse over?

@MCele MCele self-requested a review July 11, 2024 22:24
@ma7payne
Copy link
Contributor Author

TEST OK #11205

@MCele
Copy link
Contributor

MCele commented Jul 29, 2024

Mati en general la visualziación me parece que esta hermosa, dejo algunas cositas que fui viendo en una primer revisión:

  • Filtro "desde" si es obligatorio debería precargarse con un dato siempre y no hacer las búsquedas si no lo tiene.
  • Las demandas "resueltas" se ven con estado "pendiente" en el listado, considerar que no deberían verse.
  • Listado: filtro de paciente no funciona la búsqueda por número de documento.
  • Un detalle visual, me parece que los motivos a mostrar en el listado deberían agruparse y sólo mostrar los que son distintos, no repetir los que coinciden por cada ocurrencia
    .....

Mati veo la gran mayoría de los cambios, faltaron algunos a considerar, pero ya que aún no se resolvió vamos a replantear uno de los pedidos. Los que veo que faltan resolver son:

  • El historial de turnos está incompleto, sigo sin ver todos los turnos posteriores a la fecha posterior al primer registro de demanda., si no que veo todos los turnos. Pero además faltan las atenciones fuera de agenda.

  • Falta agrupar motivos en el listado y sólo mostrar los que son distintos (no repetir los que coinciden por cada ocurrencia).

Por otro lado, el filtro desde habíamos pensado en que sea obligatorio, porque el listado que existe hoy en día es muy grande, pero vamos a cambiar ese pedido por la incorporación de un scroll infinito para poder paginar los datos-

src/app/app.routing.ts Outdated Show resolved Hide resolved
@ma7payne ma7payne force-pushed the CIT-277 branch 4 times, most recently from 7123c35 to 783b82d Compare July 31, 2024 15:44
@ma7payne
Copy link
Contributor Author

Mati en general la visualziación me parece que esta hermosa, dejo algunas cositas que fui viendo en una primer revisión:

  • Filtro "desde" si es obligatorio debería precargarse con un dato siempre y no hacer las búsquedas si no lo tiene.
  • Las demandas "resueltas" se ven con estado "pendiente" en el listado, considerar que no deberían verse.
  • Listado: filtro de paciente no funciona la búsqueda por número de documento.
  • Un detalle visual, me parece que los motivos a mostrar en el listado deberían agruparse y sólo mostrar los que son distintos, no repetir los que coinciden por cada ocurrencia
    .....

Mati veo la gran mayoría de los cambios, faltaron algunos a considerar, pero ya que aún no se resolvió vamos a replantear uno de los pedidos. Los que veo que faltan resolver son:

  • El historial de turnos está incompleto, sigo sin ver todos los turnos posteriores a la fecha posterior al primer registro de demanda., si no que veo todos los turnos. Pero además faltan las atenciones fuera de agenda.
  • Falta agrupar motivos en el listado y sólo mostrar los que son distintos (no repetir los que coinciden por cada ocurrencia).

Por otro lado, el filtro desde habíamos pensado en que sea obligatorio, porque el listado que existe hoy en día es muy grande, pero vamos a cambiar ese pedido por la incorporación de un scroll infinito para poder paginar los datos-

@MCele listos los cambios. La fecha de comparación de los filtros se unificó a "horaInicio" en el campo de historial, de esta forma filtra bien los resultados. Veo los "fuera de agenda" correctamente, avisame si seguis sin ver alguno. El campo de filtro "fecha desde" es obligatorio, posee un valor unicial a una semana atrás y no busca si esta vacio.

@ma7payne ma7payne force-pushed the CIT-277 branch 3 times, most recently from 399da9e to f093f38 Compare July 31, 2024 17:43
@ma7payne
Copy link
Contributor Author

ma7payne commented Aug 1, 2024

@silviroa @MarianoCampetella Los cambios fueron realizados!

@silviroa silviroa merged commit 0564482 into master Aug 2, 2024
2 checks passed
@silviroa silviroa deleted the CIT-277 branch August 2, 2024 15:15
liquid36 pushed a commit that referenced this pull request Aug 7, 2024
# [5.146.0](v5.145.0...v5.146.0) (2024-08-07)

### Bug Fixes

* **HUDS:** Soluciona acceso a huds para No profesionales ([#3049](#3049)) ([3539695](3539695))
* **IN-608:** Censo diario- error datos de cama y paciente de internacion duplicada ([#3034](#3034)) ([1c0cf2c](1c0cf2c))

### Features

* **CIT:** Implementa mockups de demanda insatisfecha ([#3006](#3006)) ([0564482](0564482))
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.

6 participants