-
Notifications
You must be signed in to change notification settings - Fork 5
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
Conversation
There was a problem hiding this 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
-
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
-
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.
-
El botón volver sigue apareciendo en el footer, y se repite dos veces.
-
Algunos items, como por ejemplo el de vacuna sigue apareciendo con el modelo viejo. También las solicitudes vencidas.
-
A esto lo veo raro visualmente, como si le faltara un plex title.
|
.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(); |
There was a problem hiding this comment.
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); |
There was a problem hiding this comment.
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 |
There was a problem hiding this comment.
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" |
There was a problem hiding this comment.
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
<detalle-prestacion *ngIf="registro.data.class === 'plan' || registro.data.class === 'regimen' || | ||
registro.data.class === 'procedimiento' || registro.data.class === 'elementoderegistro' || registro.data.class === 'producto'" |
There was a problem hiding this comment.
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?
titulo="{{ paciente.nombre }} {{ paciente.apellido }}" subtitulo="{{ paciente | | ||
documento }}"> |
There was a problem hiding this comment.
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.
return inicio && fin ? fecha >= inicio && fecha <= fin | ||
: inicio && !fin ? fecha >= inicio | ||
: fecha <= fin; |
There was a problem hiding this comment.
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; |
There was a problem hiding this comment.
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"> --> |
There was a problem hiding this comment.
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> |
There was a problem hiding this comment.
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!
Estoy de acuerdo, me parece bien! |
51afd67
to
160e417
Compare
Quedaron revisados y corregidos los cambios! Además del agregado de algunas otras cositas a nivel UI. Quedo atento. |
7b55c45
to
e1d2022
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
d5edece
to
55040ba
Compare
@AgosLizzi Listos los cambios para evitar el error! |
d94da76
to
8490a7c
Compare
✅ TEST OK (#11035) |
0660499
to
a2499c2
Compare
c99f210
to
15a0f24
Compare
@MarianoCampetella listos los cambios para la busqueda de registros. |
Requerimiento
https://proyectos.andes.gob.ar/browse/HUDS-74
Funcionalidad desarrollada
UserStory llegó a completarse
Requiere actualizaciones en la base de datos
Requiere actualizaciones en la API
Requiere actualizaciones en andes-test-integracion