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(MISC): vista-prestacion boton editar #2137

Merged
merged 2 commits into from
Aug 5, 2021
Merged

feat(MISC): vista-prestacion boton editar #2137

merged 2 commits into from
Aug 5, 2021

Conversation

plammel
Copy link
Contributor

@plammel plammel commented Feb 18, 2021 •
edited
Loading

Requerimiento

https://proyectos.andes.gob.ar/browse/MISC-130

Funcionalidad desarrollada

  1. Botón editar prestación para vista-prestacion

UserStory llegó a completarse

Requiere actualizaciones en la base de datos

  • Si
  • No

Requiere actualizaciones en la API

  • Si
  • No

Requiere actualizaciones en andes-test-integracion

  • Si
  • No

@plammel plammel requested review from a team as code owners February 18, 2021 15:53
@liquid36
Copy link
Contributor

El concepto editar no existe en las prestaciones. Debería ser romper validación o continuar registros.
Además que no hay controles del estado de la prestación y demás.

¿De donde salio este requerimiento?

@plammel
Copy link
Contributor Author

plammel commented Feb 18, 2021

El concepto editar no existe en las prestaciones. Debería ser romper validación o continuar registros.
Además que no hay controles del estado de la prestación y demás.

¿De donde salio este requerimiento?

Surgió de la necesidad de incluir en el detalle de derivación un acceso a la prestación asociada a la misma, para cargar conceptos. https://proyectos.andes.gob.ar/browse/COM-64
image
En la propuesta actual se incluyó un botón en el detalle, pero consultando a @andes/ui-ux se resolvió que la mejor opción es incluir el botón en el plex-title de la pestación.

@liquid36
Copy link
Contributor

Bueno, cuidado entonces. Depende del estado de la prestación y si la prestación es del usuario o no.
Dos caminos: traer bien todos los controles a ese componentes o proyectar el botón con un <ng-content> y que cada modulo se responsabilice.

@plammel plammel force-pushed the MISC-130 branch 2 times, most recently from 6590ed5 to 22981de Compare February 19, 2021 15:21
@plammel
Copy link
Contributor Author

plammel commented Feb 19, 2021

Bueno, cuidado entonces. Depende del estado de la prestación y si la prestación es del usuario o no.
Dos caminos: traer bien todos los controles a ese componentes o proyectar el botón con un <ng-content> y que cada modulo se responsabilice.

Buena observación. El módulo no se puede "responsabilizar" ya que al momento de invocar a vista-prestacion solo conoce el id de la prestación, pero no al usuario autor de la misma. Una opción es que el módulo que invoca indique si quiere habilitar el botón, y vista-prestación internamente verifique si el autor es el mismo que el usuario en sesión

app/src/app/modules/rup/components/huds/vistaPrestacion.ts

Line 64 in 22981de

this.puedeEditar = this.puedeEditar && this.prestacion.createdBy.id === this.auth.usuario.id;

Con respecto al estado de la prestación, aunque se rutee a la ejecución de la prestación, si la misma esta validada, redirige a pantalla de validación de la prestación y habilita la ruptura de la validación. ¿Es correcto este comportamiento?

app/src/app/modules/rup/components/huds/vistaPrestacion.ts

Line 92 in 6590ed5

this.router.navigate(['./rup/ejecucion', this.prestacion.id]);

Por otro lado, lo que indicás que el concepto "editar" no existe para las prestaciónes, podríamos revisar el aspecto semántico y adoptar un término más apropiado para este caso.

juuliotero
juuliotero reviewed Feb 23, 2021
View reviewed changes
Copy link
Contributor

@juuliotero juuliotero 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.

El código se ve bien, habría que ver los comentarios de marian acerca del requerimiento.

@andrrrl
Copy link
Contributor

andrrrl commented Mar 5, 2021

El código se ve bien, habría que ver los comentarios de marian acerca del requerimiento.

  • Idem @juuliotero.
  • Con respecto a lo del ruteo a ejecucion, sólo se puede ir a ejecución si no está validada la prestación (aun en ejecución, o al romper la validación). Como decía @liquid36 deberías consultar el estado y rutear correctamente (si está en ejecución "continuar ejecución", si ya está validada "ver resumen"), manteniendo así coherencia semántica.
  • Por lo demás, visualmente está correcto!

@plammel
Copy link
Contributor Author

plammel commented Jun 7, 2021

Pasa a on hold hasta determinar si se avanzará en el registro de prestaciones para COM.

@plammel plammel added the En pausa Temporalmente pausado label Jun 7, 2021
@liquid36
Copy link
Contributor

liquid36 commented Aug 5, 2021

  • Se agrego ruteo automatico.
  • Control de usuario o permiso de validadación nuevo.
  • Se fixea error al ver varias prestaciones.

@liquid36 liquid36 added Aprobado and removed En pausa Temporalmente pausado has_conflicts labels Aug 5, 2021
@liquid36
Copy link
Contributor

liquid36 commented Aug 5, 2021

USUARIO: mbotta
BUILD NUMBER: 4809
CYPRESS RUN: 3803
TEST START: 2021-08-05T17:38:23.539Z
TOTAL: 400
SUCCESS: 399
FAIL: 0
SKIPPED: 1

@liquid36 liquid36 added the test ok Los test estan ok label Aug 5, 2021
@liquid36 liquid36 merged commit 2a14035 into master Aug 5, 2021
@liquid36 liquid36 deleted the MISC-130 branch August 5, 2021 17:58
liquid36 pushed a commit that referenced this pull request Aug 11, 2021
@semantic-release-bot
chore(release): 5.59.0 [skip ci]
09879de 
# [5.59.0](v5.58.0...v5.59.0) (2021-08-11)

### Bug Fixes

* **ficha-epidemiologica:** fix duplicidad sidebars buscador ([#2396](#2396)) ([52100b2](52100b2))

### Features

* **EP:** refactor de historial de ficha ([#2385](#2385)) ([e86d733](e86d733))
* **huds:** clasificacion final ficha epidemiologica ([#2381](#2381)) ([36b932d](36b932d))
* **MISC:** componente semaforo priorizacion ([#2349](#2349)) ([2578922](2578922))
* **mpi:** permite letras en nro. de identificacion de extranjero ([#2386](#2386)) ([c8055e6](c8055e6))
* **per:** descargar listado de controles ([#2390](#2390)) ([8bbb5e5](8bbb5e5))
* **rup:** vista-prestacion boton editar ([#2137](#2137)) ([2a14035](2a14035))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@juuliotero juuliotero juuliotero left review comments

@liquid36 liquid36 liquid36 approved these changes

@condorpiedra condorpiedra Awaiting requested review from condorpiedra

@palita1991 palita1991 Awaiting requested review from palita1991

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.
4 participants
@plammel @liquid36 @andrrrl @juuliotero