Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

HUDS - Implementa mockups de prestaciones #2909

Merged
merged 1 commit into from
Jul 16, 2024
Merged

HUDS - Implementa mockups de prestaciones #2909

merged 1 commit into from
Jul 16, 2024

Conversation

ma7payne
Copy link
Contributor

@ma7payne ma7payne commented Aug 31, 2023

Requerimiento

https://proyectos.andes.gob.ar/browse/HUDS-74

Funcionalidad desarrollada

  1. Se implementan los mockups para el panel de búsqueda, prestaciones en ámbito de internación, detalles de la prestación, detalle de la internación y detalle de otros conceptos.
  2. Se crea nuevo selector con slider para filtro de conceptos.
  3. Se añaden filtros para agrupar las internaciones y sus prestaciones en panel de búsqueda.
  4. Se añaden filtros para agrupar las prestaciones que no pertenecen a una internación y sus prestaciones en panel de búsqueda.
  5. Se añaden filtros para agrupar los registros del plan de indicaciones de una internación.
  6. Se añaden filtros para agrupar los registros de las prestaciones que no pertenecen una internación.
  7. Se crea nueva pantalla para detalle de registros del plan de indicaciones pertenecientes a una internación.
  8. Se crea nueva pantalla para detalle de prestaciones fuera de una internación.
  9. Se crea nueva pantalla para detalle de evoluciones y otros registros de prestaciones pertenecientes a una internación.
  10. Se crea nueva pantalla para detalle de evoluciones y otros registros de prestaciones fuera de una internación.

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

@ma7payne ma7payne requested review from a team as code owners August 31, 2023 17:11
@ma7payne ma7payne changed the title HUDS - mplementa mockups de prestaciones HUDS - Implementa mockups de prestaciones Sep 1, 2023
Copy link
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.

Mati, un par de detallitos

  1. Sobre el panel de busqueda, cuando deslizo hacia la derecha y no hay más iconos visibles, se genera este espacio (adj imagen). No se si es adrede para en un futuro agregar iconos ... si no es así me parece innecesario su visualización
    H01

  2. Mirando el panel de busqueda completo, cuando selecciono uno de los filtros y me van apareciendo lo que esta cargado, estaría bueno que aparezca un plex title con lo que se selecciona, asi le damos más contexto.
    H02

  3. El botón volver sigue apareciendo en el footer, y se repite dos veces.
    H03

  4. Algunos items, como por ejemplo el de vacuna sigue apareciendo con el modelo viejo. También las solicitudes vencidas.
    H06
    HVencida

  5. Agregar línea gris divisora
    H08

  6. A esto lo veo raro visualmente, como si le faltara un plex title.
    H05

@MarianoCampetella
Copy link
Contributor

image
Para sumar otro detalle estaría bueno que si no tiene ninguna vacuna, producto, etc estaría bueno que salga una leyenda informandole al usuario de que no hay vacunas, medicamentos, etc cargados/registrados. Nose que opina @AgosLizzi

.map(internacion => {
const prestacionesPorInternacion = prestaciones.filter(prestacion => {
const fechaIngresoValida = prestacion.fecha >= internacion.fechaIngreso;
const fechaEgresoValida = internacion.fechaEgreso ? prestacion.fecha <= internacion.fechaEgreso : prestacion.fecha <= moment().toDate();
Copy link
Contributor

Choose a reason for hiding this comment

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

Ya que estas utilizando moment() cuando haces una comparativa entre fechas se utilizan los metodos isBefore, isAfter, isSameOrBefore, etc. Te dejo acontinuacion una documentacion donde estan esas comparativas:
https://momentjs.com/docs/

registros: Object.values(grupoRegistros)
};
})
.filter(grupo => grupo.registros.length > 0);
Copy link
Contributor

Choose a reason for hiding this comment

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

Aca se podria dejar como grupo => grupo.registro.length

<div class="icono"><plex-icon class="icon" size="lg"
name="documento"></plex-icon></div>
<div>
Prestación fuera de internacion
Copy link
Contributor

Choose a reason for hiding this comment

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

Falta el acento a la ultima o de internación

<div class="mt-4">
<plex-tabs (close)="onCloseTab($event)" [activeIndex]="activeIndexPrestacion"
*ngIf="elementosRUPService.ready | async">
<plex-button *feature="'hudsExploracion'" label="Exploracion" tooltip="Exploracion Visual de HUDS"
Copy link
Contributor

Choose a reason for hiding this comment

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

Falta el acento en exploración

Comment on lines 54 to 55
<detalle-prestacion *ngIf="registro.data.class === 'plan' || registro.data.class === 'regimen' ||
registro.data.class === 'procedimiento' || registro.data.class === 'elementoderegistro' || registro.data.class === 'producto'"
Copy link
Contributor

Choose a reason for hiding this comment

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

Si no es mucha molestia se podrá armar una función para acortar el largo de la condición?

Comment on lines 7 to 8
titulo="{{ paciente.nombre }} {{ paciente.apellido }}" subtitulo="{{ paciente |
documento }}">
Copy link
Contributor

Choose a reason for hiding this comment

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

Si es posible aplicar el pipe de nombre para el paciente.

Comment on lines 116 to 118
return inicio && fin ? fecha >= inicio && fecha <= fin
: inicio && !fin ? fecha >= inicio
: fecha <= fin;
Copy link
Contributor

Choose a reason for hiding this comment

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

Utilizar las funciones isSame, isBefore, etc que se explican en la documentacion de moment()

public registro;
public prestacion;
public indicaciones;
public diasSuministro;
Copy link
Contributor

Choose a reason for hiding this comment

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

Esta variable parece estar demás ya que en la linea 80 se declara una constante con el mismo nombre.

@@ -1,4 +1,4 @@
<div class="columna prestacion-offset">
<div class="columna prestacion-offset"> -->
Copy link
Contributor

Choose a reason for hiding this comment

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

Eliminar la flecha -->

@@ -17,7 +17,7 @@ <h4 *ngIf="registro?.concepto">
<div class="datos-prestacion">
<plex-icon name="historial" class="text-info" size="xl">
</plex-icon>
<h5 class="ml-1 text-primary evoluciones">Evoluciones</h5>
<h5 class="ml-1 text-primary evoluciones">Evoluciones hola</h5>
Copy link
Contributor

Choose a reason for hiding this comment

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

Quedo un hola de alguna prueba realizada!

@MarianoCampetella MarianoCampetella added the changes requested Se solicitaron cambios label Sep 5, 2023
@AgosLizzi
Copy link
Contributor

image Para sumar otro detalle estaría bueno que si no tiene ninguna vacuna, producto, etc estaría bueno que salga una leyenda informandole al usuario de que no hay vacunas, medicamentos, etc cargados/registrados. Nose que opina @AgosLizzi

Estoy de acuerdo, me parece bien!

@ma7payne ma7payne force-pushed the HUDS-74 branch 4 times, most recently from 51afd67 to 160e417 Compare September 14, 2023 16:23
@ma7payne
Copy link
Contributor Author

Quedaron revisados y corregidos los cambios! Además del agregado de algunas otras cositas a nivel UI. Quedo atento.

@ma7payne ma7payne added changes done and removed changes requested Se solicitaron cambios labels Sep 14, 2023
@ma7payne ma7payne force-pushed the HUDS-74 branch 3 times, most recently from 7b55c45 to e1d2022 Compare September 18, 2023 17:51
@AgosLizzi AgosLizzi self-requested a review September 19, 2023 11:59
Copy link
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.

  • image

  • image

@ma7payne ma7payne force-pushed the HUDS-74 branch 2 times, most recently from d5edece to 55040ba Compare October 3, 2023 12:55
@ma7payne
Copy link
Contributor Author

ma7payne commented Oct 3, 2023

  • image
  • image

@AgosLizzi Listos los cambios para evitar el error!

@silviroa
Copy link
Contributor

silviroa commented Oct 3, 2023

USUARIO: mchiappe
BUILD NUMBER: 9724
CYPRESS RUN: 8832
TEST START: 2023-10-03T13:24:45.994Z
TOTAL: 394
SUCCESS: 383
FAIL: 10
SKIPPED: 1

@AgosLizzi AgosLizzi self-requested a review October 5, 2023 11:59
@ma7payne
Copy link
Contributor Author

ma7payne commented Jun 4, 2024

✅ TEST OK (#11035)

@MarianoCampetella
Copy link
Contributor

En hallazgos, trastornos, procedimientos y productos cuando no se encuentra ningún elemento para mostrar en el listado, si bien sale la leyenda como corresponde sigue figurando el input para buscar los elementos cuando en realidad eso solo debería salir cuando al menos existiera un elemento.
image

@MarianoCampetella MarianoCampetella added changes requested Se solicitaron cambios and removed Aprobado labels Jul 12, 2024
@ma7payne ma7payne force-pushed the HUDS-74 branch 2 times, most recently from 0660499 to a2499c2 Compare July 15, 2024 18:35
@ma7payne ma7payne added changes done changes requested Se solicitaron cambios and removed changes requested Se solicitaron cambios changes done labels Jul 15, 2024
@ma7payne ma7payne force-pushed the HUDS-74 branch 2 times, most recently from c99f210 to 15a0f24 Compare July 16, 2024 12:37
@ma7payne
Copy link
Contributor Author

En hallazgos, trastornos, procedimientos y productos cuando no se encuentra ningún elemento para mostrar en el listado, si bien sale la leyenda como corresponde sigue figurando el input para buscar los elementos cuando en realidad eso solo debería salir cuando al menos existiera un elemento. image

@MarianoCampetella listos los cambios para la busqueda de registros.

@ma7payne ma7payne added changes done and removed changes requested Se solicitaron cambios labels Jul 16, 2024
@negro89 negro89 merged commit 409e81a into master Jul 16, 2024
2 checks passed
@negro89 negro89 deleted the HUDS-74 branch July 16, 2024 13:36
liquid36 pushed a commit that referenced this pull request Jul 17, 2024
# [5.143.0](v5.142.0...v5.143.0) (2024-07-17)

### Bug Fixes

* **huds:** habilita ver internaciones de cualquier efector ([#3036](#3036)) ([b83c756](b83c756))

### Features

* **HUDS:** implementa mockups de prestaciones ([#2909](#2909)) ([409e81a](409e81a))
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.

None yet

5 participants