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.

Sign up for GitHub

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

Jump to bottom

TOP - información sidebar - historial #2896

Merged
merged 1 commit into from
Sep 1, 2023
Merged

TOP - información sidebar - historial #2896

merged 1 commit into from
Sep 1, 2023

Conversation

maring0019
Copy link
Contributor

@maring0019 maring0019 commented Jul 25, 2023 •
edited
Loading

Requerimiento

https://proyectos.andes.gob.ar/browse/TOP-154

Funcionalidad desarrollada

  1. Se agrega el nombre del efector en el historial de la solicitud tanto para las solicitudes de entrada como de salida.
  2. Se cambia el tamaño del sidebar para mejor visualización del mismo.

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

@maring0019 maring0019 requested review from a team as code owners July 25, 2023 15:28
@MarianoCampetella
Copy link
Contributor

Cuando busco una solicitud del 26/01/2023 y me fijo en el historial, se puede observar que uno de los items, al tener gran cantidad de información, ocurre que se pierde la información del efector. Trataría de buscar alguna alternativa para no se pierda ninguna información y que tampoco se solapen.
image

@MarianoCampetella MarianoCampetella added the changes requested Se solicitaron cambios label Aug 3, 2023
AgosLizzi
AgosLizzi requested changes Aug 4, 2023
View reviewed changes
Copy link
Contributor

@AgosLizzi AgosLizzi left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Mar:

  • Me sumo al pedido de @MarianoCampetella (buena observación). Supongo que esto también depende del tamaño de la pantalla ... igualmente resolvería para que sea adaptable a dif formatos.
  • Cuando lo veo desde un monitor más chico, la info se muestra así:
    image
    image

Lo cual observo que el plex item se vuelve muy enorme, y genera más scroll.

@MarianoCampetella MarianoCampetella added changes done and removed changes requested Se solicitaron cambios labels Aug 7, 2023
@silviroa
Copy link
Contributor

silviroa commented Aug 7, 2023

USUARIO: mcampetella
BUILD NUMBER: 9360
CYPRESS RUN: 8499
TEST START: 2023-08-07T12:25:07.800Z
TOTAL: 403
SUCCESS: 392
FAIL: 9
SKIPPED: 2

@MarianoCampetella MarianoCampetella added changes requested Se solicitaron cambios and removed changes done labels Aug 7, 2023
@maring0019
Copy link
Contributor Author

Cuando busco una solicitud del 26/01/2023 y me fijo en el historial, se puede observar que uno de los items, al tener gran cantidad de información, ocurre que se pierde la información del efector. Trataría de buscar alguna alternativa para no se pierda ninguna información y que tampoco se solapen. image

Se realizaron cambios.

@maring0019
Copy link
Contributor Author

Mar:

  • Me sumo al pedido de @MarianoCampetella (buena observación). Supongo que esto también depende del tamaño de la pantalla ... igualmente resolvería para que sea adaptable a dif formatos.
  • Cuando lo veo desde un monitor más chico, la info se muestra así:
    image
    image

Lo cual observo que el plex item se vuelve muy enorme, y genera más scroll.

Se realizaron cambios,

Mar:

  • Me sumo al pedido de @MarianoCampetella (buena observación). Supongo que esto también depende del tamaño de la pantalla ... igualmente resolvería para que sea adaptable a dif formatos.
  • Cuando lo veo desde un monitor más chico, la info se muestra así:
    image
    image

Lo cual observo que el plex item se vuelve muy enorme, y genera más scroll.

Se realizaron cambios.

@maring0019 maring0019 added changes done and removed changes requested Se solicitaron cambios labels Aug 8, 2023
@silviroa
Copy link
Contributor

silviroa commented Aug 8, 2023

USUARIO: mbenicio
BUILD NUMBER: 9374
CYPRESS RUN: 8511
TEST START: 2023-08-08T15:04:57.739Z
TOTAL: 371
SUCCESS: 361
FAIL: 8
SKIPPED: 2

@silviroa
Copy link
Contributor

USUARIO: mbenicio
BUILD NUMBER: 9392
CYPRESS RUN: 8529
TEST START: 2023-08-11T14:51:15.565Z
TOTAL: 344
SUCCESS: 340
FAIL: 2
SKIPPED: 2

@silviroa
Copy link
Contributor

USUARIO: mbenicio
BUILD NUMBER: 9404
CYPRESS RUN: 8541
TEST START: 2023-08-14T14:04:18.054Z
TOTAL: 403
SUCCESS: 401
FAIL: 0
SKIPPED: 2

@maring0019
Copy link
Contributor Author

image Al probarlo en una pantalla chica me ocurre lo que se muestra en la imagen. Para sumar no siempre es recomendable expandir tanto el sidebar ya que cierta información se pierde. Una posible solución, sin antes consultarla previamente, podría ser que si en un item vienen cuatro elementos ordenarlo con un grid mediante dos filas y dos columnas.

Que mal, a modificar de nuevo.

@maring0019 maring0019 force-pushed the TOP-154 branch 3 times, most recently from f71c6ee to fbbfb73 Compare August 18, 2023 17:38
@maring0019 maring0019 added changes done and removed changes requested Se solicitaron cambios labels Aug 18, 2023
@maring0019
Copy link
Contributor Author

image Al probarlo en una pantalla chica me ocurre lo que se muestra en la imagen. Para sumar no siempre es recomendable expandir tanto el sidebar ya que cierta información se pierde. Una posible solución, sin antes consultarla previamente, podría ser que si en un item vienen cuatro elementos ordenarlo con un grid mediante dos filas y dos columnas.

Que mal, a modificar de nuevo.

Se realizaron nuevos cambios.

@silviroa
Copy link
Contributor

USUARIO: mbenicio
BUILD NUMBER: 9479
CYPRESS RUN: 8606
TEST START: 2023-08-18T17:59:59.525Z
TOTAL: 403
SUCCESS: 400
FAIL: 1
SKIPPED: 2

@silviroa
Copy link
Contributor

USUARIO: mbenicio
BUILD NUMBER: 9480
CYPRESS RUN: 8607
TEST START: 2023-08-18T18:23:54.153Z
TOTAL: 385
SUCCESS: 382
FAIL: 1
SKIPPED: 2

@silviroa
Copy link
Contributor

USUARIO: mbenicio
BUILD NUMBER: 9486
CYPRESS RUN: 8613
TEST START: 2023-08-22T12:34:55.099Z
TOTAL: 403
SUCCESS: 400
FAIL: 1
SKIPPED: 2

@silviroa
Copy link
Contributor

USUARIO: mbenicio
BUILD NUMBER: 9488
CYPRESS RUN: 8615
TEST START: 2023-08-22T12:56:49.224Z
TOTAL: 403
SUCCESS: 400
FAIL: 1
SKIPPED: 2

@silviroa
Copy link
Contributor

USUARIO: mbenicio
BUILD NUMBER: 9490
CYPRESS RUN: 8617
TEST START: 2023-08-22T13:38:17.226Z
TOTAL: 403
SUCCESS: 401
FAIL: 0
SKIPPED: 2

@silviroa silviroa added test ok Los test estan ok and removed test fail labels Aug 22, 2023
@negro89 negro89 merged commit cae32ee into master Sep 1, 2023
2 checks passed
@negro89 negro89 deleted the TOP-154 branch September 1, 2023 18:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MarianoCampetella MarianoCampetella MarianoCampetella approved these changes

@negro89 negro89 negro89 approved these changes

@ma7payne ma7payne Awaiting requested review from ma7payne ma7payne is a code owner automatically assigned from andes/code-review

@AgosLizzi AgosLizzi Awaiting requested review from AgosLizzi

Assignees
No one assigned
Labels
Aprobado test ok Los test estan ok
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@maring0019 @MarianoCampetella @silviroa @negro89 @AgosLizzi