Skip to content

CIT - Visualización de historial de turnos#3186

Merged
negro89 merged 2 commits intomasterfrom
CIT-341
Jun 11, 2025
Merged

CIT - Visualización de historial de turnos#3186
negro89 merged 2 commits intomasterfrom
CIT-341

Conversation

@MarianoCampetella
Copy link
Copy Markdown
Contributor

Requerimiento

https://proyectos.andes.gob.ar/browse/CIT-341

Funcionalidad desarrollada

  1. Se hace un refactor en el tab de historial con lo siguiente:
  • Se agegan los filtros: Por estado, rango de fechas y prestación.
  • Se agrega un scroll infinito
  • Se visualizan turnos suspendidos que pertenezcan a agendas ya suspendidas o no.
  • Los turnos suspendidos que ya fueron reasignados se muestran con un hint "reasignado"
  • Se plexea todo el listado
  • Dentro del tab turnos se cambia el formato del label de "Sin turnos asignados" si el paciente no tiene turnos a futuro

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

@MarianoCampetella MarianoCampetella requested review from a team as code owners May 14, 2025 16:03
Copy link
Copy Markdown
Contributor

@negro89 negro89 left a comment

Choose a reason for hiding this comment

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

Campe, le vino bien la plexeada (quedó excelente 👌🏼) pero hay algunos detalles de funcionalidad:

  1. No hay control sobre el uso de los filtros, permitiendo por ejemplo poner en el caso de las fechas una mayor en desde y menor en hasta sin que el usuario pueda ver el error.
  2. Esta filtrando raro; al filtrar por fechas trae mas cantidad de resultados que sin filtrar, incluso los resultados cambian si se usa un solo filtro fecha.

Dejo video con aclaraciones en cada caso: https://www.loom.com/share/4892bb24d0924cf3b5119efab91f0fe4?sid=56f554fb-d297-45f8-a4b7-cb7928dbdd09

</plex-wrapper>
<div *ngIf="!(ultimosTurnos$ | async)?.length" justify="center">
<plex-label class="flex-column" icon="turno-bold" type="info" size="lg" direction="column"
titulo="No se han encontrado turnos registrados para el paciente." style="margin-top: 4em;">
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Ya que tenemos una hoja de estilos prexistente, debería ir ahi (buena práctica). Otra opción es aprovechar las clases de bootstrap asi no tenemos tantos estilos desperdigados.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Corregido!


<div *ngIf="!turnosPaciente || turnosPaciente?.length === 0" justify="center">
<plex-label class="flex-column" icon="turno-bold" type="info" size="lg" direction="column"
titulo="El paciente no presenta turnos asignados." style="margin-top: 4em;">
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Idem al anterior: lo ideal sería calse de bootstrap 👍🏼

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Corregido, todo estilo de boostrap utilizado se agrego en una hoja de estilo existente!

@negro89 negro89 added the changes requested Se solicitaron cambios label May 15, 2025
Copy link
Copy Markdown
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.

Campe, según lo charlado te dejo comentario:

Ampliar el sidebar de tal manera que la pantalla quede 50/50 (50% main y 50% sidebar), para mejorar la visualización del historial de turnos

image
vista desde notebook.. Fijate que cuando realizas el scroll, se activa un scroll horizontal

@MarianoCampetella
Copy link
Copy Markdown
Contributor Author

2. Esta filtrando raro; al filtrar por fechas trae mas cantidad de resultados que sin filtrar, incluso los resultados cambian si se usa un solo filtro fecha.

Corregido, para el punto uno se agrego un control de tal forma que no te permita poner una fechaDesde mayor a una fechaHasta. En cuanto al punto dos ahora filtra como corresponde y como yapa se corrigió un problema que existía con el scroll infinito.

@MarianoCampetella
Copy link
Copy Markdown
Contributor Author

Campe, según lo charlado te dejo comentario:

Ampliar el sidebar de tal manera que la pantalla quede 50/50 (50% main y 50% sidebar), para mejorar la visualización del historial de turnos

image vista desde notebook.. Fijate que cuando realizas el scroll, se activa un scroll horizontal

El sidebar anteriormente se encontraba ocupando un 30% del margen de la pantalla con lo cual ahora se aplio un 40% logrando que toda la información que se visualiza se vea perfectamente. Si es muy necesario un 50% se cambia a ese tamaño.
En cuanto a lo otro no le logrado obtener un segundo scroll dentro de la pantalla de mi notebook y en pantallas un poco mas grande se ve perfectamente. Cualquier inconveniente lo vemos entre los dos a ver que tipo de pantalla ocurre y que otra solución se le puede buscar.

@MarianoCampetella MarianoCampetella added changes done and removed changes requested Se solicitaron cambios labels May 19, 2025
@Fabio-Ramirez
Copy link
Copy Markdown
Contributor

Compa, sabes que noto que no me trae algunos turnos a "futuros", por ejemplo en el dia de la fecha no veo un turrno de la semana que viene, paso captura:
image
image

@Fabio-Ramirez Fabio-Ramirez added changes requested Se solicitaron cambios and removed changes done labels May 30, 2025
@negro89
Copy link
Copy Markdown
Contributor

negro89 commented May 30, 2025

Campe, el historial quedó de 10. Esta bien que no muestre los turnos a futuro @Fabio-Ramirez pero al parecer tenemos un problema en el tab turnos: acá si deberían verse los turnos que el paciente tiene de hoy en adelante y no está sucediendo.
Esto sería lo único que resta para que terminar.

@MarianoCampetella
Copy link
Copy Markdown
Contributor Author

Campe, el historial quedó de 10. Esta bien que no muestre los turnos a futuro @Fabio-Ramirez pero al parecer tenemos un problema en el tab turnos: acá si deberían verse los turnos que el paciente tiene de hoy en adelante y no está sucediendo. Esto sería lo único que resta para que terminar.

Corregido! Ahora se visualiza el turno en el tab "turnos". También se hizo un pequeño cambio en la api!

@MarianoCampetella MarianoCampetella added changes done and removed changes requested Se solicitaron cambios labels Jun 2, 2025
@negro89 negro89 merged commit b54d731 into master Jun 11, 2025
2 checks passed
@negro89 negro89 deleted the CIT-341 branch June 11, 2025 17:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants