feat(IN): mapa-camas, visualizar camas agrupadas por sector #2145

waltergarrido · 2021-02-23T20:57:44Z

Requerimiento

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

Funcionalidad desarrollada

se crea el listado de recursos , se agrupa las camas por sectores.

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

liquid36 · 2021-02-23T21:05:43Z

USUARIO: wgarrido
BUILD NUMBER: 3433
CYPRESS RUN: 2528
TEST START: 2021-02-23T21:04:37.064Z
TOTAL: 358
SUCCESS: 357
FAIL: 0
SKIPPED: 1

plammel

Cambiar el título del PR por uno más descriptivo.

plammel · 2021-02-24T13:59:28Z

src/app/apps/rup/mapa-camas/mapa-camas.routing.ts

+
+

Suggested change

plammel · 2021-02-24T14:13:26Z

src/app/apps/rup/mapa-camas/views/mapa-camas-capa/mapa-camas-capa.component.ts

+    verListadoRecursos() {
+        this.listadoRecursos = this.listadoRecursos ? false : true;
+

+    }


Suggested change

verListadoRecursos() {

this.listadoRecursos = this.listadoRecursos ? false : true;

}

verListadoRecursos() {

this.listadoRecursos = !this.listadoRecursos;

}

Sugerencia: un nombre más apropiado para esta función podría ser toggleListadoRecursos

O quizás se pueda cambiar el valor en el template, ya que no se realiza otra acción además de esta.

plammel · 2021-02-24T14:15:27Z

...app/apps/rup/mapa-camas/views/mapa-camas-capa/recursos-listado/recursos-listado.component.ts

+
+
+
+
+


Suggested change

plammel · 2021-02-24T14:16:16Z

...app/apps/rup/mapa-camas/views/mapa-camas-capa/recursos-listado/recursos-listado.component.ts

+
+

Suggested change

@plammel corregido

JSantarelli

El botón que switchea entre 'Listado de camas' y 'Listado de recursos' impacta directamente (y exclusivamente) sobre el listado. Quizás la ubicación de este botón sería más adecuada al nivel del <plex-title> del listado.

waltergarrido · 2021-02-26T12:30:41Z

El botón que switchea entre 'Listado de camas' y 'Listado de recursos' impacta directamente (y exclusivamente) sobre el listado. Quizás la ubicación de este botón sería más adecuada al nivel del <plex-title> del listado.

@JSantarelli corregido

liquid36

Falta que la cama quede como seleccionada.

Primera review

liquid36 · 2021-03-02T17:07:43Z

...p/apps/rup/mapa-camas/views/mapa-camas-capa/recursos-listado/recursos-listado.component.html

+<plex-accordion *ngFor="let sector of sectore$| async ">
+    <plex-panel>
+        <div plex-accordion-title justify>
+            <section class="w-50">


Quitar este
. Solo hace que el texto se caiga a una segunda linea.

liquid36 · 2021-03-02T17:07:56Z

...app/apps/rup/mapa-camas/views/mapa-camas-capa/recursos-listado/recursos-listado.component.ts

+                return arreglo.map(sector => {
+
+                    return {
+                        nombre: sector.sectorName.split(',').reverse().join(','),


Suggested change

nombre: sector.sectorName.split(',').reverse().join(','),

nombre: sector.sectorName.split(',').reverse().join(', '),

liquid36 · 2021-03-02T17:12:58Z

...app/apps/rup/mapa-camas/views/mapa-camas-capa/recursos-listado/recursos-listado.component.ts

+                const arreglo = [];
+                snapshots = snapshots.filter(snap => snap.estado !== 'inactiva');
+                snapshots.map(c => {
+                    if (!(arreglo.length > 0)) {


Esta condición no tiene sentido, con el find es mas que suficiente.

@liquid36 corregido

liquid36 · 2021-03-02T17:19:37Z

...p/apps/rup/mapa-camas/views/mapa-camas-capa/recursos-listado/recursos-listado.component.html

+                </plex-label>
+            </section>
+        </div>
+        <plex-grid size="md" type="full">


Suggested change

<plex-grid size="md" type="full">

<plex-grid size="md" cols="3">

liquid36 · 2021-03-02T17:32:54Z

Los botones de la cama no funcionan.

JSantarelli · 2021-03-03T11:18:36Z

Viendo el prototipo noté que falta la información secundaria en los cabezales de los accordions. La idea es que el usuario pueda previsualizar los recursos alojados en el sector antes de desplegar. Lo ven posible/útil?

liquid36 · 2021-03-03T11:24:58Z

Esta @JSantarelli pero todo en una misma linea.

JSantarelli · 2021-03-03T11:46:55Z

Ah perfecto @liquid36, entonces el pedido sería desacoplar de allí esa info.

JSantarelli · 2021-03-05T10:46:14Z

...p/apps/rup/mapa-camas/views/mapa-camas-capa/recursos-listado/recursos-listado.component.html

+    <plex-panel>
+        <div plex-accordion-title justify>
+            <section class="w-50">
+                <plex-label size="lg" case="capitalize" titulo="{{ sector.nombre }}" icon="hospital">


Falta agregar subtitulo con los recursos que aloja el sector (actualmente todo en una misma línea en el título).

@JSantarelli corregido

liquid36

Haría el icono de la cama más grande
Sacaría el icono del paciente (ya hay uno)
quitaría el nombre del sector de cada cama.
En el headear del acordion veo muy desproporcionado el tamañan del icono contra el titulo principal.

@JSantarelli Qué opinas?

liquid36 · 2021-03-12T14:18:13Z

No se puede deseleccionar una cama, para poder cambiar la hora del mapa.

waltergarrido · 2021-03-12T16:24:22Z

No se puede deseleccionar una cama, para poder cambiar la hora del mapa.

@liquid36 corregido

JSantarelli

Luego de internar un paciente, se cierran automáticamente todos los accordions. El usuario pierde foco y quizás hay probabilidades de que quiera seguir realizando operaciones sobre el paciente recientemente internado. La sugerencia es que el foco quede en el paciente internado (accordion correspondiente abierto y sidebar con detalle del paciente en cuestión).

internacion.mp4

# [5.42.0](v5.41.0...v5.42.0) (2021-04-14) ### Bug Fixes * **mpi:** revalidacion de paciente ([#2183](#2183)) ([d2c4cb8](d2c4cb8)) * **mpi:** se corrige error al mostrar badge "sin dni" en pacietes validados menores a 5 años ([#2172](#2172)) ([bf77ec2](bf77ec2)) * **virtual-scroll:** corrige variables generales ([#2171](#2171)) ([9d71238](9d71238)) ### Features * **ficha-epidemiologica:** nuevos campos ficha covid ([5759752](5759752)) * **internacion:** notifica ruta volver a mapa de camas ([#2207](#2207)) ([ac0b323](ac0b323)) * **mapa-camsa:** visualizar camas agrupadas por sector ([#2145](#2145)) ([81ff32b](81ff32b)) * **pdp:** al refrescar la pantalla mantener sesión abierta ([#2209](#2209)) ([ac7dd18](ac7dd18)) * **pdp:** visualizar mis laboratorios ([#2188](#2188)) ([9cd3708](9cd3708)) * **pdp:** visualizar mis turnos actuales ([#2191](#2191)) ([bb6d44b](bb6d44b)) * **pdp:** visualizar mis vacunas ([#2187](#2187)) ([fdddf15](fdddf15))

waltergarrido requested review from a team as code owners February 23, 2021 20:57

waltergarrido requested review from MCele and negro89 February 23, 2021 20:57

liquid36 added the test ok Los test estan ok label Feb 23, 2021

plammel suggested changes Feb 24, 2021

View reviewed changes

negro89 approved these changes Feb 24, 2021

View reviewed changes

JSantarelli reviewed Feb 26, 2021

View reviewed changes

waltergarrido requested review from JSantarelli and plammel February 26, 2021 12:27

waltergarrido changed the title ~~In 252~~ feat(IN):mapa-camas, visualizar camas agrupadas por sector Feb 26, 2021

waltergarrido force-pushed the IN-252 branch from b3278eb to f63d5c3 Compare February 26, 2021 12:32

liquid36 changed the title ~~feat(IN):mapa-camas, visualizar camas agrupadas por sector~~ feat(IN): mapa-camas, visualizar camas agrupadas por sector Mar 2, 2021

liquid36 requested changes Mar 2, 2021

View reviewed changes

liquid36 added changes requested Se solicitaron cambios test ok Los test estan ok and removed test ok Los test estan ok labels Mar 2, 2021

liquid36 reviewed Mar 2, 2021

View reviewed changes

JSantarelli suggested changes Mar 5, 2021

View reviewed changes

waltergarrido force-pushed the IN-252 branch from 394f795 to 1e67569 Compare March 11, 2021 11:36

waltergarrido requested review from JSantarelli and liquid36 March 11, 2021 11:37

liquid36 reviewed Mar 11, 2021

View reviewed changes

waltergarrido requested a review from liquid36 March 12, 2021 13:12

JSantarelli reviewed Mar 29, 2021

View reviewed changes

liquid36 force-pushed the IN-252 branch from 486bf2e to cbfd3a3 Compare April 5, 2021 12:02

waltergarrido added 7 commits April 12, 2021 14:32

refactor(IN):probando la visualizacion de algunos datos, falta mucho

2452e5c

feat(IN)visualizar camas agrupadas por sector

6afd1a1

feat(IN):visualizar camas agrupadas por sector

6c76999

(IN): se corrige posicion de boton y espacios

7360293

(IN):correcciones

c371123

(IN) cambios visuales sugeridos

c1741cf

(IN) deseleccionar cama y volver a resumen

958edee

liquid36 force-pushed the IN-252 branch from cbfd3a3 to f383cf0 Compare April 13, 2021 11:34

liquid36 approved these changes Apr 13, 2021

View reviewed changes

fix(mapa-camas): iconos de vistas

46efd25

liquid36 force-pushed the IN-252 branch from f383cf0 to 46efd25 Compare April 13, 2021 12:40

liquid36 merged commit 81ff32b into master Apr 13, 2021

liquid36 deleted the IN-252 branch April 13, 2021 12:57

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat(IN): mapa-camas, visualizar camas agrupadas por sector #2145

feat(IN): mapa-camas, visualizar camas agrupadas por sector #2145

waltergarrido commented Feb 23, 2021

liquid36 commented Feb 23, 2021

plammel left a comment

plammel Feb 24, 2021

plammel Feb 24, 2021

negro89 Feb 24, 2021

plammel Feb 24, 2021

plammel Feb 24, 2021

waltergarrido Feb 26, 2021

JSantarelli left a comment •

edited

Loading

waltergarrido commented Feb 26, 2021

liquid36 left a comment

liquid36 Mar 2, 2021

liquid36 Mar 2, 2021

liquid36 Mar 2, 2021

waltergarrido Mar 11, 2021

liquid36 Mar 2, 2021

liquid36 commented Mar 2, 2021

JSantarelli commented Mar 3, 2021

liquid36 commented Mar 3, 2021

JSantarelli commented Mar 3, 2021

JSantarelli Mar 5, 2021

waltergarrido Mar 11, 2021

liquid36 left a comment

liquid36 commented Mar 12, 2021

waltergarrido commented Mar 12, 2021

JSantarelli left a comment

	nombre: sector.sectorName.split(',').reverse().join(','),
	nombre: sector.sectorName.split(',').reverse().join(', '),

	<plex-grid size="md" type="full">
	<plex-grid size="md" cols="3">

feat(IN): mapa-camas, visualizar camas agrupadas por sector #2145

feat(IN): mapa-camas, visualizar camas agrupadas por sector #2145

Conversation

waltergarrido commented Feb 23, 2021

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

liquid36 commented Feb 23, 2021

plammel left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

JSantarelli left a comment • edited Loading

Choose a reason for hiding this comment

waltergarrido commented Feb 26, 2021

liquid36 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

liquid36 commented Mar 2, 2021

JSantarelli commented Mar 3, 2021

liquid36 commented Mar 3, 2021

JSantarelli commented Mar 3, 2021

Choose a reason for hiding this comment

Choose a reason for hiding this comment

liquid36 left a comment

Choose a reason for hiding this comment

liquid36 commented Mar 12, 2021

waltergarrido commented Mar 12, 2021

JSantarelli left a comment

Choose a reason for hiding this comment

JSantarelli left a comment •

edited

Loading