Skip to content

HUDS - Separa la visualización del los cdas de guardias#3347

Merged
negro89 merged 4 commits into
masterfrom
HUDS-195
May 21, 2026
Merged

HUDS - Separa la visualización del los cdas de guardias#3347
negro89 merged 4 commits into
masterfrom
HUDS-195

Conversation

@nicolasarana
Copy link
Copy Markdown
Contributor

Requerimiento

HUDS-195

Funcionalidad desarrollada

  1. Agrega filtro y separa la visualización de los CDAs de guardias.
  2. Se agrega botones de regenerar cdas y recargar el listado

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

  • Si
  • No

@nicolasarana nicolasarana requested review from a team as code owners May 4, 2026 18:12

<ng-container
*ngIf="filtroActual !== 'solicitudes' && filtroActual !== 'planes' && filtroActual !== 'laboratorios' && filtroActual !== 'vacunas' && filtroActual !== 'dominios'">
*ngIf="filtroActual !== 'solicitudes' && filtroActual !== 'planes' && filtroActual !== 'laboratorios' && filtroActual !== 'vacunas' && filtroActual !== 'dominios' && filtroActual !== 'guardias'">
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.

Compa se podria simplificar la vista, colocando en un metodo el analisis de estas multiples condiciones.

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.

Hola Fabi, listo ya se hizo el cambio gracias.

<ng-container *ngIf="filtroActual === 'guardias'">
<plex-loader *ngIf="disabledBtnCDA" class="sidebar-loader" type="ball-pulse"></plex-loader>

<div *ngIf="guardias && guardias.length > 0" class="conceptos hover list-unstyled mt-1">
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.

Se puede simplificar, por *ngIf="guardias?.length".

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.

listo :D

@Fabio-Ramirez
Copy link
Copy Markdown
Contributor

Compa, noto el icono guardia con un color neutro y tamaño diferente a los demas...
image

@Fabio-Ramirez Fabio-Ramirez added the changes requested Se solicitaron cambios label May 5, 2026
@nicolasarana
Copy link
Copy Markdown
Contributor Author

Compa, noto el icono guardia con un color neutro y tamaño diferente a los demas... image

Corregido

@nicolasarana nicolasarana added changes done and removed changes requested Se solicitaron cambios labels May 5, 2026
@Fabio-Ramirez
Copy link
Copy Markdown
Contributor

Fabio-Ramirez commented May 8, 2026

Compa, estan bastantes bien los cambios, lo unico que en el color del icon se ve distinto al borde y numero...
image
image

Tambien noto que el color de las pestañas no es el mismo al Guardias
image

@nicolasarana nicolasarana force-pushed the HUDS-195 branch 3 times, most recently from a174bff to 584ac7c Compare May 8, 2026 17:46
@nicolasarana nicolasarana force-pushed the HUDS-195 branch 2 times, most recently from 436b469 to 5d9036d Compare May 11, 2026 14:58
@nicolasarana
Copy link
Copy Markdown
Contributor Author

Compa, estan bastantes bien los cambios, lo unico que en el color del icon se ve distinto al borde y numero... image image

Tambien noto que el color de las pestañas no es el mismo al Guardias image

Corregido

@nicolasarana nicolasarana added falta 1 revisor! Ponele voluntad asi mergeamos and removed changes done labels May 18, 2026
@Fabio-Ramirez Fabio-Ramirez added Aprobado and removed falta 1 revisor! Ponele voluntad asi mergeamos labels May 18, 2026
@negro89 negro89 merged commit 4e393d5 into master May 21, 2026
1 check passed
@negro89 negro89 deleted the HUDS-195 branch May 21, 2026 18:33
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