IN - Separar visualización camas censables y no censables by nicolasarana · Pull Request #3225 · andes/app

nicolasarana · 2025-09-02T17:59:25Z

Requerimiento

IN-644

Funcionalidad desarrollada

Separar en el sidebar con el estado de camas la visualización de camas censables y no censables.
Por defecto debe mostrar los totales solo considerando camas censables
Agregar un slide que diga "Incluir No censables" y al estar en true, sume las camas no censables

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

… censables"

silviroa

La idea no era cambiar el filtro existente, si no que por defecto muestra solo camas censables y el slide para incluir censable se debe agregar en el Titulo del sidebar donde se ven los totales de camas

silviroa · 2025-09-05T18:59:16Z

-                camasFiltradas = camasFiltradas.filter((snap: ISnapshot) => !snap.esCensable);
-            } else if (esCensable.id === 1) {
-                camasFiltradas = camasFiltradas.filter((snap: ISnapshot) => snap.esCensable);
+        if (esCensable != null) {


No hace falta poner explicitamente el != null

MarianoCampetella · 2025-09-08T18:40:03Z

+        if (this.filtro.censable !== undefined && this.filtro.censable !== null) {
+            this.mapaCamasService.esCensable.next(this.filtro.censable.id);
+        } else {
+            this.mapaCamasService.esCensable.next(this.mostrarTodasCamas ? null : 1);
+        }


Estas dos condiciones las podemos remplazar por una ternaria para que quede mas legible a la hora de interpretarlo.

this.mapaCamasService.esCensable.next(this.filtro.censable?.id ?? (this.mostrarTodasCamas ? null : 1));

MarianoCampetella · 2025-09-08T18:41:29Z


+    onCensableChange() {
+        if (this.filtro.censable) {
+            this.mostrarTodasCamas = false;


Me queda la duda en que momento la variable this.mostrarTodasCamas se puede convertir en true ya que en la declaración esta como falsa y si la condición del if se cumple o no, sigue siendo falsa.

El plex-bool (switch) solo aparece cuando no hay nada elegido en el select de censable (*ngIf="!filtro.censable").
Si la persona elige “Censable” o “No Censable”, el switch desaparece y onCensableChange() se encarga de resetear su valor.
Esto evita que quede un valor del switch activo al mismo tiempo que una opción en el select, lo cual aplicaría mal el filtro.

silviroa · 2025-09-26T18:39:45Z

Si aplican filtro por censable, modificar titulo en el sidebar y ocultar slide

… del sidebar"

nicolasarana · 2025-09-29T14:21:27Z

Si aplican filtro por censable, modificar titulo en el sidebar y ocultar slide

(Ya se corrigió lo recomendado)

Fabio-Ramirez · 2025-10-16T13:38:01Z

Nico, por lo general esta correcta la tarea y sus correcciones... por otro lado, con lo que estuvimos viendo recien en conjunto con @AgosLizzi, dejo comentado que: en capa profesional, hay un caso que no se ven todas las camas no censables, dejo imagen...

hay 3 camas no censables, sin embargo en el sidebar se muestran 2 en total.
En cambio en capa estadística, esta correcto el contador.

Compa, efectivamente hay q hacer la correccion por este caso.

nicolasarana · 2025-11-07T19:07:21Z

Nico, por lo general esta correcta la tarea y sus correcciones... por otro lado, con lo que estuvimos viendo recien en conjunto con @AgosLizzi, dejo comentado que: en capa profesional, hay un caso que no se ven todas las camas no censables, dejo imagen... hay 3 camas no censables, sin embargo en el sidebar se muestran 2 en total. En cambio en capa estadística, esta correcto el contador.

Compa, efectivamente hay q hacer la correccion por este caso.

Se corrigió el bug de profesional con camas censables

AgosLizzi

Nico, un detalle super pequeño. Cuando aplicamos el filtro por censable o no censable cambia el plex title del sidebar y queda una linea abajo solo la hora, siempre mirando la pantalla desde una notebook. En estos casos se podría achicar a md el título para que no quede esa diferencia?
Solo cambiaría de tamaño cuando se aplica el filtro

nicolasarana · 2025-11-13T17:38:25Z

Nico, un detalle super pequeño. Cuando aplicamos el filtro por censable o no censable cambia el plex title del sidebar y queda una linea abajo solo la hora, siempre mirando la pantalla desde una notebook. En estos casos se podría achicar a md el título para que no quede esa diferencia? Solo cambiaría de tamaño cuando se aplica el filtro

Ya se realizo el cambio. Saludos

feat(IN-644):"Separar visualización de estado de camas censables y no…

d8b282b

… censables"

nicolasarana requested review from a team as code owners September 2, 2025 17:59

nicolasarana requested review from Silgabgar and aldoEMatamala September 2, 2025 17:59

nicolasarana changed the title ~~feat(IN-644):"Separar visualización de estado de camas censables y no…~~ feat(IN-644):"Separar visualización camas censables y no censables Sep 2, 2025

silviroa requested changes Sep 5, 2025

View reviewed changes

silviroa added the changes requested Se solicitaron cambios label Sep 5, 2025

nicolasarana added changes done and removed changes requested Se solicitaron cambios labels Sep 8, 2025

Se deja funcionando tanto el select como el slide

e7d0553

nicolasarana force-pushed the IN-644 branch from 5f1438b to e7d0553 Compare September 8, 2025 17:11

MarianoCampetella requested changes Sep 8, 2025

View reviewed changes

MarianoCampetella added changes requested Se solicitaron cambios and removed changes done labels Sep 8, 2025

nicolasarana added changes done and removed changes requested Se solicitaron cambios labels Sep 9, 2025

nicolasarana added 2 commits September 9, 2025 16:00

Se cambia la condición por una ternaria

fc095f0

style(IN):"Se cambio el slide de lugar"

4498895

feat(INT):"Se agrega funcionalidad en el select y cambios en el title…

eb57b84

… del sidebar"

Fabio-Ramirez added the Consenso Requiere acordar algunos criterios label Oct 16, 2025

Fabio-Ramirez added changes requested Se solicitaron cambios and removed changes done Consenso Requiere acordar algunos criterios labels Oct 31, 2025

feat(IN):"Se arregla bugs de cama no censables en profesionales-médicos"

f68ed3e

nicolasarana removed the changes requested Se solicitaron cambios label Nov 7, 2025

nicolasarana added the changes done label Nov 7, 2025

Fabio-Ramirez approved these changes Nov 11, 2025

View reviewed changes

AgosLizzi reviewed Nov 13, 2025

View reviewed changes

style(IN):Cambios en el sidebar y en el title

69a34a6

AgosLizzi self-requested a review November 13, 2025 18:19

AgosLizzi approved these changes Nov 13, 2025

View reviewed changes

MarianoCampetella added changes requested Se solicitaron cambios and removed changes done labels Dec 5, 2025

MarianoCampetella changed the title ~~feat(IN-644):"Separar visualización camas censables y no censables~~ IN - Separar visualización camas censables y no censables Dec 5, 2025

MarianoCampetella approved these changes Dec 5, 2025

View reviewed changes

nicolasarana added changes done and removed changes requested Se solicitaron cambios labels Jan 8, 2026

fix : Se soluciona error con salas y filtros

f687589

Conversation

nicolasarana commented Sep 2, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Requerimiento

Funcionalidad desarrollada

UserStory llegó a completarse

Requiere actualizaciones en la base de datos

Requiere actualizaciones en la API

Requiere actualizaciones en andes-test-integracion

Uh oh!

silviroa left a comment

Choose a reason for hiding this comment

Uh oh!

silviroa Sep 5, 2025

Choose a reason for hiding this comment

Uh oh!

nicolasarana Sep 10, 2025

Choose a reason for hiding this comment

Uh oh!

MarianoCampetella Sep 8, 2025

Choose a reason for hiding this comment

Uh oh!

nicolasarana Sep 9, 2025

Choose a reason for hiding this comment

Uh oh!

MarianoCampetella Sep 8, 2025

Choose a reason for hiding this comment

Uh oh!

nicolasarana Sep 9, 2025

Choose a reason for hiding this comment

Uh oh!

silviroa commented Sep 26, 2025

Uh oh!

nicolasarana commented Sep 29, 2025

Uh oh!

Fabio-Ramirez commented Oct 16, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

nicolasarana commented Nov 7, 2025

Uh oh!

AgosLizzi left a comment

Choose a reason for hiding this comment

Uh oh!

nicolasarana commented Nov 13, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

nicolasarana commented Sep 2, 2025 •

edited

Loading

Fabio-Ramirez commented Oct 16, 2025 •

edited

Loading