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

RUP - Implementa mockups en punto de inicio #2945

Merged
merged 1 commit into from
Jun 25, 2024
Merged

RUP - Implementa mockups en punto de inicio #2945

merged 1 commit into from
Jun 25, 2024

Conversation

ma7payne
Copy link
Contributor

@ma7payne ma7payne commented Nov 3, 2023

Requerimiento

https://proyectos.andes.gob.ar/browse/RUP-370

Funcionalidad desarrollada

  1. Implementa rediseño basado en mockup para punto de inicio
  2. Implementa rediseño basado en mockup para selección de pacientes en agenda dinámica

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

@ma7payne ma7payne requested review from a team as code owners November 3, 2023 16:12
@ma7payne ma7payne force-pushed the RUP-370 branch 4 times, most recently from 3671d4c to 41f0c3b Compare November 7, 2023 13:41
@silviroa
Copy link
Contributor

silviroa commented Nov 7, 2023

USUARIO: mchiappe
BUILD NUMBER: 9878
CYPRESS RUN: 8985
TEST START: 2023-11-07T16:39:25.899Z
TOTAL: 352
SUCCESS: 275
FAIL: 76
SKIPPED: 1

@silviroa
Copy link
Contributor

silviroa commented Nov 8, 2023

USUARIO: mchiappe
BUILD NUMBER: 9881
CYPRESS RUN: 8988
TEST START: 2023-11-08T18:06:55.587Z
TOTAL: 326
SUCCESS: 301
FAIL: 23
SKIPPED: 2
⚠️⚠️⚠️⚠️
Hay archivos que no corrieron. Vuelva a intentar.

@silviroa
Copy link
Contributor

USUARIO: mchiappe
BUILD NUMBER: 9886
CYPRESS RUN: 8993
TEST START: 2023-11-10T11:52:06.069Z
TOTAL: 403
SUCCESS: 335
FAIL: 66
SKIPPED: 2

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, te dejo un par de cositas que vi:

  1. Los botones cambian de visual cuando cambio de pantalla. Fijate también que en el mismo video cuando quiero abrir un listado "fuera de agenda" no me deja, recien cuando lo abrí desde otro dif, me deja pasar al que quiero. te dejo video: https://www.loom.com/share/1e8b2faa976a4e858552ca29969a52ec +
    image

  2. Por fa agregar la línea gris divisora
    image

  3. Re ordenar tooltip hacia la izquierda, no importa que se superponga con los botones.
    image

  4. No carga bien las agendas, por alguna razón cuando cambio de pantalla, me refresca y ahi carga las que corresponde. Te dejo video: https://www.loom.com/share/683ed0805b064cae81c68194be23c2f0

@ma7payne ma7payne force-pushed the RUP-370 branch 2 times, most recently from 431db80 to e724e4b Compare November 16, 2023 13:04
@ma7payne
Copy link
Contributor Author

Mati, te dejo un par de cositas que vi:

  1. Los botones cambian de visual cuando cambio de pantalla. Fijate también que en el mismo video cuando quiero abrir un listado "fuera de agenda" no me deja, recien cuando lo abrí desde otro dif, me deja pasar al que quiero. te dejo video: https://www.loom.com/share/1e8b2faa976a4e858552ca29969a52ec +
    image
  2. Por fa agregar la línea gris divisora
    image
  3. Re ordenar tooltip hacia la izquierda, no importa que se superponga con los botones.
    image
  4. No carga bien las agendas, por alguna razón cuando cambio de pantalla, me refresca y ahi carga las que corresponde. Te dejo video: https://www.loom.com/share/683ed0805b064cae81c68194be23c2f0

Listos los cambios!

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, van correcciones:

  1. Hay varios badge que estan mal acomodados o quedaron fuera de contextos. Ye adjunto capturas:
  • badgeRUP

  • 0badge rup

  • image

Si vas al mockup los badge estan en otra posición, no se si fue adrede cambiarlos o simplemente quedaron así

  1. Te adjunto un video donde sucede algo parecido a la primera observación, pero con el botón, cuando voy a "registrar inasistencia", el botón de "revertir cambios" quedá en un lugar donde no debería. https://www.loom.com/share/c31658f8f48e4b05bbf07f4601be6603
  2. Este tooltip, que paso? Observar también el botón, me parece que no es correcto.
    rup tooltip
  3. Cuando doy un sobreturno, queda así. Acomodar situación con botones y agregar plex title de sobreturnos, por favor.
    Rup sobreturnos
  4. La acción de autocitado esta mal. Fijate que en demo esta bien la implementación... Como primera instancia en este caso el botón de "iniciar prestación" esta de más.
    Rup autocitado

@ma7payne ma7payne force-pushed the RUP-370 branch 2 times, most recently from 52478e8 to 997a3ec Compare December 6, 2023 17:03
@ma7payne
Copy link
Contributor Author

ma7payne commented Dec 6, 2023

Mati, van correcciones:

  1. Hay varios badge que estan mal acomodados o quedaron fuera de contextos. Ye adjunto capturas:
  • badgeRUP
  • 0badge rup
  • image

Si vas al mockup los badge estan en otra posición, no se si fue adrede cambiarlos o simplemente quedaron así

  1. Te adjunto un video donde sucede algo parecido a la primera observación, pero con el botón, cuando voy a "registrar inasistencia", el botón de "revertir cambios" quedá en un lugar donde no debería. https://www.loom.com/share/c31658f8f48e4b05bbf07f4601be6603
  2. Este tooltip, que paso? Observar también el botón, me parece que no es correcto.
    rup tooltip
  3. Cuando doy un sobreturno, queda así. Acomodar situación con botones y agregar plex title de sobreturnos, por favor.
    Rup sobreturnos
  4. La acción de autocitado esta mal. Fijate que en demo esta bien la implementación... Como primera instancia en este caso el botón de "iniciar prestación" esta de más.
    Rup autocitado

@AgosLizzi Estan listos los nuevos cambios!

@AgosLizzi AgosLizzi self-requested a review December 7, 2023 15:34
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, van nuevos cambios:

  1. Hay un problema con el scroll y el label. Desde mi opinión, no se si es necesario el doble scroll, el que va sobre el listado ocasiona muchos problemas. Y luego el label queda como estático. Va video adjunto https://www.loom.com/share/0daf14ccbcb84e098dc11c606a265120
  2. En sobreturno los badge se repiten y el botón queda mal ubicado
    sobreturno
  3. Consulta, hay posibilidad que los badge queden uno al lado del otro y no uno sobre el otro?
    badge00
  4. Y por último, muy parecido al ejemplo 1, desde autocitado sucede lo siguiente:
    https://www.loom.com/share/2d0c44a14e3f47c9a25e55dd84014e55
    *El principio del video, yo me confundí y empecé a grabar en demo, desestimalo por fa.

@ma7payne
Copy link
Contributor Author

ma7payne commented Jun 5, 2024

✅ TEST OK (#11041)

@negro89 negro89 added Aprobado publicar and removed prioritario Tarea con prioridad alta changes done labels Jun 5, 2024
@negro89
Copy link
Contributor

negro89 commented Jun 5, 2024

Agregar tooltip faltante para cerrar el sidebar

Copy link
Contributor

@MCele MCele left a comment

Choose a reason for hiding this comment

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

Mati se ve hermosa la nueva pantalla :D
Te comento por aca lo que estuvimos charlando que falta revisar:

No se visualizan los fuera de agenda algunos dias (parece ser que es cuando no hay ninguna agenda para visualizar).
Cuando se visualizan los fuera de agenda debería ser un sólo item para todos los fuera de agenda cargados en el día, en cambio ahora se visualiza uno por cada prestacion cargada.
Uploading image.png…

Dato de color aprovecharía a sacar el campo requerido en el selec de prestacion, ya que nunca se aplica como tal

@ma7payne ma7payne added changes requested Se solicitaron cambios and removed publicar Aprobado labels Jun 13, 2024
@ma7payne ma7payne force-pushed the RUP-370 branch 2 times, most recently from 130fc21 to 1a45df1 Compare June 13, 2024 16:23
@ma7payne
Copy link
Contributor Author

@MCele Listos los últimos cambios en la visualización del listado de agendas.

@ma7payne ma7payne added changes done and removed changes requested Se solicitaron cambios labels Jun 13, 2024
@ma7payne ma7payne force-pushed the RUP-370 branch 2 times, most recently from 421e9da to 83b7469 Compare June 14, 2024 12:21
@ma7payne
Copy link
Contributor Author

Test OK #11094

@negro89
Copy link
Contributor

negro89 commented Jun 14, 2024

Mati, quedó joya excepto por un par de detallitos visuales que se encontraron. Estas serían las ultimas tres cositas a solucionar antes de subirlo. Va con imagenes:

  1. Cuando seleccionamos una agenda con prestacion no nominalizada desde el gestor, el sidebar nos muestra información repetida.
    nominalizada x 2

  2. La botonera de sobreturnos esta ordenada inversamente con respecto a la de turnos. Además se sugirió que el ícono para registrar la inasistencia sea adi-account-cruz para no confundir con las notas.
    botones al verrez

  3. Por ultimo, un inconveniente más que se encontró relacionado al punto 2 es que al registrar inasistencia de la tercer agenda (por ejemplo) se recarga el sidebar y queda mostrando los detalles de la primera agenda del listado; para lo cual se propone, además de mantener en el sidebar la información de la agenda seleccionada, resaltar esta agenda en el listado al igual que en el gestor.
    Imagen de la información que se muestra seguido a registrar inasistencia en la 3er agenda del listado:
    foco agenda

@ma7payne
Copy link
Contributor Author

image

Mati, quedó joya excepto por un par de detallitos visuales que se encontraron. Estas serían las ultimas tres cositas a solucionar antes de subirlo. Va con imagenes:

  1. Cuando seleccionamos una agenda con prestacion no nominalizada desde el gestor, el sidebar nos muestra información repetida.
    nominalizada x 2
  2. La botonera de sobreturnos esta ordenada inversamente con respecto a la de turnos. Además se sugirió que el ícono para registrar la inasistencia sea adi-account-cruz para no confundir con las notas.
    botones al verrez
  3. Por ultimo, un inconveniente más que se encontró relacionado al punto 2 es que al registrar inasistencia de la tercer agenda (por ejemplo) se recarga el sidebar y queda mostrando los detalles de la primera agenda del listado; para lo cual se propone, además de mantener en el sidebar la información de la agenda seleccionada, resaltar esta agenda en el listado al igual que en el gestor.
    Imagen de la información que se muestra seguido a registrar inasistencia en la 3er agenda del listado:
    foco agenda

@negro89 Listos los cambios en los botones y en la actualización de la vista para evitar el cambio de agenda (puntos 2 y 3). El punto numero 1 corresponde a una vista del gestor de agendas. Quedo atento!

@ma7payne ma7payne added changes done and removed changes requested Se solicitaron cambios labels Jun 24, 2024
@negro89 negro89 merged commit 82dd4a1 into master Jun 25, 2024
2 checks passed
@negro89 negro89 deleted the RUP-370 branch June 25, 2024 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changes done test ok Los test estan ok
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants