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

feat(HUDS): Agregar filtro de descargas pendientes #2668

Merged
merged 6 commits into from
May 12, 2023
Merged

feat(HUDS): Agregar filtro de descargas pendientes #2668

merged 6 commits into from
May 12, 2023

Conversation

leon-bravo
Copy link
Contributor

Requerimiento

MISC-228 - Agregar filtro a la pantalla de descargas pendientes

Funcionalidad desarrollada

  1. Se cambio el tamaño del titulo y de los iconos para que sean mas estéticos
  2. Se agregaron filtros por fecha desde y hasta, que inician con la fecha del día de hoy

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

  • Si
  • No

@leon-bravo leon-bravo requested review from a team as code owners June 8, 2022 14:20
plammel
plammel suggested changes Jun 9, 2022
View reviewed changes
Copy link
Contributor

@plammel plammel left a comment

Choose a reason for hiding this comment

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

Cuando se busca por un día en particular, la búsqueda no arroja resultados. https://www.loom.com/share/7e4169a508bd47288ae9859afc51baad
Por otro lado, si se filtra por fecha estaría bueno mostrar la fecha de la descarga en el detalle de la misma

@leon-bravo leon-bravo changed the title Misc 228 feat(HUDS): Agregar filtro de descargas pendientes Jun 9, 2022
@leon-bravo leon-bravo requested a review from plammel June 9, 2022 14:43
@leon-bravo
Copy link
Contributor Author

Cuando se busca por un día en particular, la búsqueda no arroja resultados. https://www.loom.com/share/7e4169a508bd47288ae9859afc51baad Por otro lado, si se filtra por fecha estaría bueno mostrar la fecha de la descarga en el detalle de la misma

Si tenes razón que es mas informativo, ahí lo agrego

@leon-bravo
Copy link
Contributor Author

Cuando se busca por un día en particular, la búsqueda no arroja resultados. https://www.loom.com/share/7e4169a508bd47288ae9859afc51baad Por otro lado, si se filtra por fecha estaría bueno mostrar la fecha de la descarga en el detalle de la misma

Si tenes razón que es mas informativo, ahí lo agrego

@leon-bravo leon-bravo closed this Jun 10, 2022
@leon-bravo
Copy link
Contributor Author

leon-bravo commented Jun 10, 2022 •
edited
Loading

ya quedaron listos los cambios

@leon-bravo leon-bravo reopened this Jun 10, 2022
@martinebucarey
Copy link
Contributor

USUARIO: lbravo
BUILD NUMBER: 6629
CYPRESS RUN: 5483
TEST START: 2022-06-16T12:29:40.489Z
TOTAL: 409
SUCCESS: 401
FAIL: 7
SKIPPED: 1

@martinebucarey
Copy link
Contributor

USUARIO: lbravo
BUILD NUMBER: 6645
CYPRESS RUN: 5499
TEST START: 2022-06-22T13:14:10.925Z
TOTAL: 406
SUCCESS: 400
FAIL: 5
SKIPPED: 1

@martinebucarey
Copy link
Contributor

USUARIO: lbravo
BUILD NUMBER: 6718
CYPRESS RUN: 5571
TEST START: 2022-06-30T12:14:00.081Z
TOTAL: 353
SUCCESS: 338
FAIL: 14
SKIPPED: 1

@martinebucarey
Copy link
Contributor

Ahora que estas con lo de rxjs es el mismo comentario que te hice antes, es mas el next que cambiaste por la subscripcion al evento es para laburar así.

martinebucarey
martinebucarey previously requested changes Jul 28, 2022
View reviewed changes
Copy link
Contributor

@martinebucarey martinebucarey left a comment

Choose a reason for hiding this comment

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

Ahora que estas con lo de rxjs es el mismo comentario que te hice antes, es mas el next que cambiaste por la subscripcion al evento es para laburar así.

@silviroa silviroa added the changes requested Se solicitaron cambios label Aug 30, 2022
@MarianoCampetella MarianoCampetella removed the changes requested Se solicitaron cambios label Aug 31, 2022
@MarianoCampetella
Copy link
Contributor

Resta resolver si se dejan los cambios que estan ahora o se vuelve al commit anterior. Los test ya estan resueltos!

@negro89 negro89 dismissed martinebucarey’s stale review May 10, 2023 18:09

operario deprecado

@silviroa
Copy link
Contributor

USUARIO: lmlagos
BUILD NUMBER: 8868
CYPRESS RUN: 8035
TEST START: 2023-05-10T18:39:14.399Z
TOTAL: 393
SUCCESS: 386
FAIL: 5
SKIPPED: 2

@silviroa
Copy link
Contributor

USUARIO: lmlagos
BUILD NUMBER: 8869
CYPRESS RUN: 8036
TEST START: 2023-05-10T18:57:24.112Z
TOTAL: 335
SUCCESS: 325
FAIL: 8
SKIPPED: 2

@silviroa
Copy link
Contributor

USUARIO: mcampetella
BUILD NUMBER: 8871
CYPRESS RUN: 8038
TEST START: 2023-05-11T12:26:54.142Z
TOTAL: 406
SUCCESS: 404
FAIL: 0
SKIPPED: 2

@silviroa silviroa added test ok Los test estan ok and removed test fail labels May 11, 2023
@negro89 negro89 merged commit 3c194e3 into master May 12, 2023
@negro89 negro89 deleted the MISC-228 branch May 12, 2023 17:11
liquid36 pushed a commit that referenced this pull request May 17, 2023
@semantic-release-bot
chore(release): 5.119.0 [skip ci]
cfe3ba8 
# [5.119.0](v5.118.0...v5.119.0) (2023-05-17)

### Bug Fixes

* **IN-537:** Error al editar egreso ([#2861](#2861)) ([5e48b8e](5e48b8e))
* **TOP:** se marca la solicitud seleccionada ([#2840](#2840)) ([1f6946e](1f6946e))

### Features

* **HUDS:** Agregar filtro de descargas pendientes  ([#2668](#2668)) ([3c194e3](3c194e3))
* **HUDS:** modifica muestra en info de prestacion ([#2859](#2859)) ([fe4dadb](fe4dadb))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AgosLizzi AgosLizzi AgosLizzi approved these changes

@negro89 negro89 negro89 approved these changes

@palita1991 palita1991 palita1991 approved these changes

@plammel plammel plammel approved these changes

@martinebucarey martinebucarey martinebucarey left review comments

Assignees

@MCele MCele

@MarianoCampetella MarianoCampetella

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

Successfully merging this pull request may close these issues.
9 participants
@leon-bravo @martinebucarey @silviroa @MarianoCampetella @plammel @palita1991 @negro89 @AgosLizzi @MCele