PRIV - Visualizar motivos de acceso a la HUDS #2958

aldoEMatamala · 2023-11-28T17:25:54Z

Requerimiento

https://proyectos.andes.gob.ar/browse/PRIV-65

Funcionalidad desarrollada

Se reemplazan los motivos de acceso a la HUDS estáticos por accesos a BD.

Acceso explícito a la HUDS (modal).
Buscador de Turnos y prestaciones.
Al iniciar una prestación.
Desde la internación (modal) de acceso a los registros y plan de indicaciones.
Solicitudes: bandeja de entrada/salida al iniciar una prestación.

UserStory llegó a completarse

Si
No
No corresponde

Requiere actualizaciones en la base de datos

Si ----> Crear colección llamada "motivosHuds" agregada a BD DEMO
No

Requiere actualizaciones en la API

Requiere actualizaciones en andes-test-integracion

silviroa · 2023-12-07T15:12:52Z

USUARIO: amatamala
BUILD NUMBER: 10035
CYPRESS RUN: 9144
TEST START: 2023-12-07T14:37:40.604Z
TOTAL: 399
SUCCESS: 333
FAIL: 64
SKIPPED: 2

src/app/modules/rup/components/huds/modal-motivo-acceso-huds.component.ts

silviroa · 2023-12-20T15:22:15Z

USUARIO: mcampetella
BUILD NUMBER: 10092
CYPRESS RUN: 9200
TEST START: 2023-12-20T14:47:03.531Z
TOTAL: 360
SUCCESS: 289
FAIL: 69
SKIPPED: 2

silviroa · 2023-12-20T19:00:34Z

USUARIO: mcampetella
BUILD NUMBER: 10102
CYPRESS RUN: 9210
TEST START: 2023-12-20T18:23:57.909Z
TOTAL: 406
SUCCESS: 346
FAIL: 58
SKIPPED: 2

JuanIRamirez · 2023-12-22T11:49:09Z

Al continuar una prestación no toma el motivo de acceso a la HUDS de la DB.

JuanIRamirez · 2023-12-22T11:57:42Z

En el modal la opción 'Intervención en el proceso de cuidado del paciente' se solapa en 2 lineas, esto se corrige en 'modal-motivo-acceso-huds.html' (linea 7) -div class="w-60"-

aldoEMatamala · 2024-01-05T18:10:01Z

En el modal la opción 'Intervención en el proceso de cuidado del paciente' se solapa en 2 lineas, esto se corrige en 'modal-motivo-acceso-huds.html' (linea 7) -div class="w-60"-

corregido! gracias !

negro89

Aldinho, habría que hacer unos pequeños cambios en el esquema de este nuevo recurso:

Agregar una columna "key" para cada motivo
Renombrar el atributo "motivo" a nombre/label que es mas representativo para como venimos usando internamente los componentes que sugieren opciones
El atributo "descripcion" contendría texto a modo informativo para orientar al usuario (Después vemos qué, por ahora no lo utilizariamos)
El atributo "moduloDefault" debería ser un array, para no repetir registros en caso de compartir motivo por defecto entre varios modulos

Por otro lado, almacenar del lado de la app el motivo y el detalle en un array puede ser un poco confuso en cuanto a recordar en qué posición se encuentra cada elemento o para el caso de los undefined. Sugeriría utilizar un objeto similar a
motivo: { key: string, detalle: string }

Cualquier cosita quedo a disposición

aldoEMatamala · 2024-07-04T11:50:43Z

aldoEMatamala · 2024-07-04T11:53:46Z

Aldinho, habría que hacer unos pequeños cambios en el esquema de este nuevo recurso:
1. Agregar una columna "key" para cada motivo

2. Renombrar el atributo "motivo" a nombre/label que es mas representativo para como venimos usando internamente los componentes que sugieren opciones

3. El atributo "descripcion" contendría texto a modo informativo para orientar al usuario (Después vemos qué, por ahora no lo utilizariamos)

4. El atributo "moduloDefault" debería ser un array, para no repetir registros en caso de compartir motivo por defecto entre varios modulos
Por otro lado, almacenar del lado de la app el motivo y el detalle en un array puede ser un poco confuso en cuanto a recordar en qué posición se encuentra cada elemento o para el caso de los undefined. Sugeriría utilizar un objeto similar a motivo: { key: string, detalle: string }

Cualquier cosita quedo a disposición

buendi lautaro!
Quedan subidas las modificaciones de esquema como se charlo por slack. Se crea interfaz en app (IMotivosAcceso) para los objetos motivo: { key: string, detalle: string }. Se hacen las modificaciones a los casos de test. Así quedan los registros en demo.

seleccion-manual para los motivos del modal
arreglo para multiples modulos con el mismo motivo
cambio de nombres
motivo-> key
descripcion-> label
descripcion (nuevo)
cualquier cosa lo seguimos viendo.Saludos!

negro89

Holi, funciona de 10 pero dejé algunas observaciones. Trato de resumirlas en ...

En relación al pasaje del motivo entre el modal y los otros componentes, noté que a veces se lo trata como un objeto (uso correcto) y otras veces como valores separados.
En todos los casos la búsqueda del motivo por defecto (moduloDefault) está perfecto.
No me quería extender mucho, cualquier cosa despejamos charlando 👌🏼

negro89 · 2024-07-17T15:42:37Z

src/app/components/top/solicitudes/solicitudes.component.ts

- preAccesoHuds(motivoAccesoHuds) {
- const motivo = (typeof motivoAccesoHuds !== 'string') ? motivoAccesoHuds[0] : motivoAccesoHuds;
+ preAccesoHuds(motivoAccesoHuds: IMotivoAcceso | string) {
+ const motivo = (typeof motivoAccesoHuds !== 'string') ? motivoAccesoHuds.motivo : motivoAccesoHuds;


En este caso, cuando se cancela el modal para seleccionar motivo, entra con valor null y muestra error por consola

Agregado el control!

negro89 · 2024-07-17T15:45:47Z

src/app/modules/rup/components/ejecucion/hudsBusquedaPaciente.component.ts

 profesional: this.auth.profesional ? this.auth.profesional : null,
 idTurno: null,
 idPrestacion: null,
- detalleMotivo: motivoAccesoHuds[1]
+ detalleMotivo: motivoAccesoHuds.descripcionAcceso


Esta propiedad no debería enviarse. El uso correcto es enviar el objeto completo en la prop. "motivo" de la linea 85. De esta manera podemos estandarizar su uso y no varía en cada componente.

negro89 · 2024-07-17T15:51:34Z

src/app/modules/rup/components/ejecucion/puntoInicio.component.ts

@@ -912,7 +929,7 @@ export class PuntoInicioComponent implements OnInit, OnDestroy {
 profesional: this.auth.profesional,
 idTurno: this.accesoHudsTurno,
 idPrestacion: this.accesoHudsPrestacion,
- detalleMotivo: (typeof motivoAccesoHuds !== 'string') ? motivoAccesoHuds[1] : ''
+ detalleMotivo: (typeof motivoAccesoHuds !== 'string') ? motivoAccesoHuds.descripcionAcceso : null


misma observación que en componente anterior. Debería pasarse el objeto completo en "motivo" para estandarizar su uso.

Se mantienen las dos interfaces como se charlo. Saludos!

negro89 · 2024-07-17T16:02:13Z

src/app/modules/rup/components/huds/modal-motivo-acceso-huds.component.ts

 motivoSelect() {
 return this.motivoSelected === null;
 }

 notificarAccion(flag: boolean) {
 if (flag) {
 const item = this.motivosAccesoHuds.find((elem) => elem.id === this.motivoSelected);
- this.motivoAccesoHuds.emit([item.label, this.detalleMotivo]);
+ this.motivo = {


Acá si no se pasa el objeto completo, al menor debería respetarse las keys de la interfaz key y label (nuevamente para estandarizar su uso y que no varía de componente a componente).

Si el problema es identificar los valores por separado en el html, se puede acceder haciendo [(ngModel)]="motivo.label"

Se mantienen las dos interfaces como se charlo. Saludos!

aldoEMatamala requested review from a team as code owners November 28, 2023 17:25

aldoEMatamala requested review from negro89 and AgosLizzi November 28, 2023 17:25

aldoEMatamala force-pushed the PRIV-65 branch 2 times, most recently from 2d47dea to e3f2f75 Compare November 28, 2023 18:57

aldoEMatamala force-pushed the PRIV-65 branch 2 times, most recently from 4c63956 to 311982a Compare December 7, 2023 14:25

silviroa added the test fail label Dec 7, 2023

MarianoCampetella requested changes Dec 20, 2023

View reviewed changes

src/app/modules/rup/components/huds/modal-motivo-acceso-huds.component.ts Outdated Show resolved Hide resolved

MarianoCampetella added the changes requested Se solicitaron cambios label Dec 20, 2023

MarianoCampetella force-pushed the PRIV-65 branch from 311982a to 98a88bb Compare December 20, 2023 14:20

silviroa added test fail and removed test fail labels Dec 20, 2023

JuanIRamirez self-requested a review December 22, 2023 11:51

github-actions bot added the has_conflicts label Dec 22, 2023

aldoEMatamala force-pushed the PRIV-65 branch 2 times, most recently from 6101dc7 to 31d0c9a Compare January 4, 2024 18:41

github-actions bot removed the has_conflicts label Jan 5, 2024

aldoEMatamala force-pushed the PRIV-65 branch from 31d0c9a to 258943a Compare January 5, 2024 18:11

aldoEMatamala requested a review from a team as a code owner January 5, 2024 18:11

aldoEMatamala force-pushed the PRIV-65 branch from 258943a to bdec23d Compare January 5, 2024 18:16

negro89 added the publicar label May 31, 2024

negro89 self-requested a review May 31, 2024 18:05

negro89 requested changes May 31, 2024

View reviewed changes

negro89 added changes requested Se solicitaron cambios and removed publicar changes done labels May 31, 2024

negro89 self-assigned this Jun 3, 2024

github-actions bot added the has_conflicts label Jun 3, 2024

aldoEMatamala force-pushed the PRIV-65 branch from 51bfc10 to 4fb262e Compare June 26, 2024 15:18

aldoEMatamala removed the has_conflicts label Jun 26, 2024

github-actions bot added the has_conflicts label Jun 28, 2024

aldoEMatamala force-pushed the PRIV-65 branch from 4fb262e to 99e14cd Compare July 1, 2024 18:46

aldoEMatamala mentioned this pull request Jul 3, 2024

PRIV - Visualizar motivos de acceso a la HUDS andes/api#1918

Open

4 tasks

aldoEMatamala force-pushed the PRIV-65 branch 4 times, most recently from 35feb59 to 996e9cc Compare July 3, 2024 15:18

aldoEMatamala added changes done and removed has_conflicts changes requested Se solicitaron cambios labels Jul 3, 2024

aldoEMatamala requested a review from negro89 July 4, 2024 11:53

github-actions bot added the has_conflicts label Jul 16, 2024

negro89 reviewed Jul 17, 2024

View reviewed changes

aldoEMatamala force-pushed the PRIV-65 branch from 996e9cc to afa25d3 Compare July 18, 2024 18:26

feat(PRIV-65): Visualizar motivos de acceso a la HUDS

ec6c23b

aldoEMatamala force-pushed the PRIV-65 branch from afa25d3 to ec6c23b Compare July 18, 2024 18:35

aldoEMatamala requested a review from negro89 July 18, 2024 18:40

aldoEMatamala removed the has_conflicts label Jul 18, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

PRIV - Visualizar motivos de acceso a la HUDS #2958

PRIV - Visualizar motivos de acceso a la HUDS #2958

aldoEMatamala commented Nov 28, 2023 •

edited

Loading

silviroa commented Dec 7, 2023

silviroa commented Dec 20, 2023

silviroa commented Dec 20, 2023

JuanIRamirez commented Dec 22, 2023

JuanIRamirez commented Dec 22, 2023 •

edited

Loading

aldoEMatamala commented Jan 5, 2024

negro89 left a comment •

edited

Loading

aldoEMatamala commented Jul 4, 2024

aldoEMatamala commented Jul 4, 2024 •

edited

Loading

negro89 left a comment

negro89 Jul 17, 2024

aldoEMatamala Jul 18, 2024

negro89 Jul 17, 2024

negro89 Jul 17, 2024

aldoEMatamala Jul 18, 2024

negro89 Jul 17, 2024

aldoEMatamala Jul 18, 2024

PRIV - Visualizar motivos de acceso a la HUDS #2958

Are you sure you want to change the base?

PRIV - Visualizar motivos de acceso a la HUDS #2958

Conversation

aldoEMatamala commented Nov 28, 2023 • edited Loading

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

silviroa commented Dec 7, 2023

silviroa commented Dec 20, 2023

silviroa commented Dec 20, 2023

JuanIRamirez commented Dec 22, 2023

JuanIRamirez commented Dec 22, 2023 • edited Loading

aldoEMatamala commented Jan 5, 2024

negro89 left a comment • edited Loading

Choose a reason for hiding this comment

aldoEMatamala commented Jul 4, 2024

aldoEMatamala commented Jul 4, 2024 • edited Loading

negro89 left a comment

Choose a reason for hiding this comment

negro89 Jul 17, 2024

Choose a reason for hiding this comment

aldoEMatamala Jul 18, 2024

Choose a reason for hiding this comment

negro89 Jul 17, 2024

Choose a reason for hiding this comment

negro89 Jul 17, 2024

Choose a reason for hiding this comment

aldoEMatamala Jul 18, 2024

Choose a reason for hiding this comment

negro89 Jul 17, 2024

Choose a reason for hiding this comment

aldoEMatamala Jul 18, 2024

Choose a reason for hiding this comment

aldoEMatamala commented Nov 28, 2023 •

edited

Loading

JuanIRamirez commented Dec 22, 2023 •

edited

Loading

negro89 left a comment •

edited

Loading

aldoEMatamala commented Jul 4, 2024 •

edited

Loading