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 - Revisar y analizar comportamiento del botón continuar #2903

Merged
merged 1 commit into from
Feb 14, 2024
Merged

TOP - Revisar y analizar comportamiento del botón continuar #2903

merged 1 commit into from
Feb 14, 2024

Conversation

aldoEMatamala
Copy link
Contributor

@aldoEMatamala aldoEMatamala commented Aug 11, 2023 •
edited
Loading

Requerimiento

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

Funcionalidad desarrollada

  1. Corregidos badges de solicitudes de "Turno dado" a "Ejecución"
  2. Corregidos filtros de "Turno dado" que también mostraba solicitudes en "Ejecución"
  3. Botón "Continuar registro" restringido a solicitudes donde el usuario tiene permisos para rup e inició el registro de atención medica o tiene permisos especiales, siempre con con la prestación en ejecución.

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

@aldoEMatamala aldoEMatamala requested review from a team as code owners August 11, 2023 14:04
@MarianoCampetella
Copy link
Contributor

image
En las solicitudes de salida encontré el siguiente problema (muy raro):
Al seleccionar el dropdown se abre pero al parecer no existe ninguna acción para poder seleccionar. Solamente me ocurrió con esa solicitud, con el resto que tiene el mismo estado no ocurre. En demo se puede visualizar la opción de "Continuar Registro".

@MarianoCampetella MarianoCampetella added the changes requested Se solicitaron cambios label Aug 14, 2023
@aldoEMatamala aldoEMatamala added changes done and removed changes requested Se solicitaron cambios labels Sep 28, 2023
@aldoEMatamala
Copy link
Contributor Author

ero al parecer no existe ninguna acción para poder seleccionar. Solamente me ocurrió con esa solicitud, con el resto que tiene el mismo estado no ocurre. En demo se puede visualizar la opción de "Continuar Registro".

hola campe! gracias por la corrección, quedo arreglado este asunto

@silviroa
Copy link
Contributor

USUARIO: amatamala
BUILD NUMBER: 9699
CYPRESS RUN: 8811
TEST START: 2023-09-28T15:57:50.702Z
TOTAL: 356
SUCCESS: 347
FAIL: 7
SKIPPED: 2

@silviroa
Copy link
Contributor

USUARIO: amatamala
BUILD NUMBER: 9700
CYPRESS RUN: 8812
TEST START: 2023-09-28T16:09:49.722Z
TOTAL: 403
SUCCESS: 401
FAIL: 0
SKIPPED: 2

@silviroa silviroa added test ok Los test estan ok and removed test fail labels Sep 28, 2023
@MarianoCampetella
Copy link
Contributor

MarianoCampetella commented Dec 7, 2023 •
edited by aldoEMatamala
Loading

image

Me queda la duda con el botón "continuar registros" ya que en la descripcion de la tarea dice:
El botón "continuar registro" para una solicitud ... en la de salida nunca debería verse. Las puedo ver para las solicitudes con "turno dado" o "ejecución"

Ya que estamos corregir los conflictos

Aldo:
Corregido gracias!

@MarianoCampetella MarianoCampetella added changes requested Se solicitaron cambios and removed changes done labels Dec 7, 2023
@negro89 negro89 added the prioritario Tarea con prioridad alta label Dec 19, 2023
@MarianoCampetella
Copy link
Contributor

MarianoCampetella commented Dec 20, 2023 •
edited by aldoEMatamala
Loading

image

También se observa que desde la bandeja de salida, a partir de diciembre del 2022 existen varias solicitudes con distintos estados los cuales en el dropdown sigue figurando "continuar registro".
A demás, en la bandeja de salida cuando filtro por Turno dado se observa que los badges no tienen el mismo color

Aldo:
Corregido los pipes de botones y solicitud

JuanIRamirez
JuanIRamirez reviewed Dec 21, 2023
View reviewed changes
src/app/components/top/solicitudes/solicitudes.html
@@ -159,7 +159,8 @@
{{prestacion.solicitud.registros[0].valor.solicitudPrestacion.prioridad}}
</plex-badge>
<plex-badge type="{{prestacion | estadoPrestacion}}"
*ngIf="!prestacion.solicitud.turno && prestacion.estadoActual.tipo !== 'rechazada' && prestacion.estadoActual.tipo !== 'auditoria' && prestacion.estadoActual.tipo !== 'validada' && prestacion.estadoActual.tipo !== 'vencida'">
*ngIf="prestacion.estadoActual.tipo !== 'rechazada' && prestacion.estadoActual.tipo !== 'auditoria' && prestacion.estadoActual.tipo !== 'validada' && prestacion.estadoActual.tipo !== 'vencida'">
Copy link
Contributor

Choose a reason for hiding this comment

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

No es que esté mal, pero por buenas prácticas (si se puede), cuando son muchas condiciones, incluirlas en una función.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

corregido gracias!

@MCele MCele added changes requested Se solicitaron cambios and removed Aprobado changes done labels Jan 12, 2024
@aldoEMatamala aldoEMatamala added changes done and removed changes requested Se solicitaron cambios labels Feb 6, 2024
@silviroa
Copy link
Contributor

silviroa commented Feb 6, 2024

USUARIO: amatamala
BUILD NUMBER: 10447
CYPRESS RUN: 9552
TEST START: 2024-02-06T15:23:48.199Z
TOTAL: 374
SUCCESS: 364
FAIL: 7
SKIPPED: 3

@silviroa silviroa added test fail and removed test ok Los test estan ok labels Feb 6, 2024
@silviroa
Copy link
Contributor

silviroa commented Feb 6, 2024

USUARIO: amatamala
BUILD NUMBER: 10449
CYPRESS RUN: 9554
TEST START: 2024-02-06T15:50:35.161Z
TOTAL: 402
SUCCESS: 396
FAIL: 3
SKIPPED: 3

@silviroa
Copy link
Contributor

silviroa commented Feb 6, 2024

USUARIO: amatamala
BUILD NUMBER: 10451
CYPRESS RUN: 9556
TEST START: 2024-02-06T16:29:29.188Z
TOTAL: 394
SUCCESS: 391
FAIL: 0
SKIPPED: 3

@silviroa silviroa added test ok Los test estan ok and removed test fail labels Feb 6, 2024
@negro89 negro89 merged commit 63c0e6f into master Feb 14, 2024
2 checks passed
@negro89 negro89 deleted the TOP-142 branch February 14, 2024 15:54
liquid36 pushed a commit that referenced this pull request Feb 21, 2024
@semantic-release-bot
chore(release): 5.133.0 [skip ci]
c8b6a5b 
# [5.133.0](v5.132.2...v5.133.0) (2024-02-21)

### Bug Fixes

* **IN:** visualizacion de registros en sidebar ([#2973](#2973)) ([#2903](#2903)) ([63c0e6f](63c0e6f))

### Features

* **IN:** request con boton de busqueda ([#2960](#2960)) ([7454faf](7454faf))
* **TYP:** listado de prestaciones no nominalizadas ([#2962](#2962)) ([fe145ff](fe145ff))
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

@JuanIRamirez JuanIRamirez Awaiting requested review from JuanIRamirez

@ma7payne ma7payne Awaiting requested review from ma7payne

@MarianoCampetella MarianoCampetella Awaiting requested review from MarianoCampetella

Assignees

@ma7payne ma7payne

@AgosLizzi AgosLizzi

Labels
Aprobado prioritario Tarea con prioridad alta test ok Los test estan ok
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
9 participants
@aldoEMatamala @MarianoCampetella @silviroa @ma7payne @negro89 @AgosLizzi @JuanIRamirez @MCele @Fabio-Ramirez