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

IN - Eliminar indicación en estado borrador #2954

Merged
merged 1 commit into from
Mar 22, 2024
Merged

IN - Eliminar indicación en estado borrador #2954

merged 1 commit into from
Mar 22, 2024

Conversation

MarianoCampetella
Copy link
Contributor

Requerimiento

https://proyectos.andes.gob.ar/browse/IN-583

Funcionalidad desarrollada

  1. Se agrega un boton para eliminar las indicaciones que se encuentran en estado "borrador"
  2. NOTA: en el html de plan-indicaciones.component.html solamente se modifica las lineas 125 a 128 donde se agrega dicho botón.

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

@MarianoCampetella MarianoCampetella requested review from a team as code owners November 22, 2023 16:28
@silviroa
Copy link
Contributor

USUARIO: mcampetella
BUILD NUMBER: 9946
CYPRESS RUN: 9053
TEST START: 2023-11-22T16:40:49.299Z
TOTAL: 403
SUCCESS: 401
FAIL: 0
SKIPPED: 2

@silviroa silviroa added the test ok Los test estan ok label Nov 22, 2023
@silviroa
Copy link
Contributor

silviroa commented Jan 3, 2024

USUARIO: mcampetella
BUILD NUMBER: 10143
CYPRESS RUN: 9251
TEST START: 2024-01-03T13:18:08.316Z
TOTAL: 404
SUCCESS: 400
FAIL: 2
SKIPPED: 2

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

silviroa commented Jan 3, 2024

USUARIO: mcampetella
BUILD NUMBER: 10149
CYPRESS RUN: 9257
TEST START: 2024-01-03T14:47:15.628Z
TOTAL: 406
SUCCESS: 404
FAIL: 0
SKIPPED: 2

@silviroa silviroa added test ok Los test estan ok and removed test fail labels Jan 3, 2024
@MarianoCampetella MarianoCampetella added the muy prioritario Tarea con prioridad muy alta label Jan 9, 2024
@MarianoCampetella MarianoCampetella mentioned this pull request Jan 12, 2024
Merged
9 tasks
@MCele MCele added prioritario Tarea con prioridad alta and removed muy prioritario Tarea con prioridad muy alta labels Feb 1, 2024
negro89
negro89 requested changes Feb 29, 2024
View reviewed changes
Copy link
Contributor

@negro89 negro89 left a comment

Choose a reason for hiding this comment

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

Anda bien. El unico problema es que si al eliminar la indicación el sidebar con los detalles se encuentra abierto, se puede seguir visualizando los detalles e incluso editarla y guardar, obteniendo un toast de confirmación. Esto no debería suceder. Dejo videito.

https://app.screencastify.com/v3/watch/IxvXPlP7yaWfVTcKcpcK

@negro89 negro89 added the changes requested Se solicitaron cambios label Feb 29, 2024
@MarianoCampetella
Copy link
Contributor Author

Anda bien. El unico problema es que si al eliminar la indicación el sidebar con los detalles se encuentra abierto, se puede seguir visualizando los detalles e incluso editarla y guardar, obteniendo un toast de confirmación. Esto no debería suceder. Dejo videito.

https://app.screencastify.com/v3/watch/IxvXPlP7yaWfVTcKcpcK

Corregido, ahora cada vez que se elimina una indicación en estado draft se cierra el sidebar en caso de estar abierto. También se corrigió un detalle del título al momento de ver el detalle de una indicación.

@MarianoCampetella MarianoCampetella added changes done and removed changes requested Se solicitaron cambios labels Mar 7, 2024
Fabio-Ramirez
Fabio-Ramirez reviewed Mar 7, 2024
View reviewed changes
src/app/apps/rup/mapa-camas/views/plan-indicaciones/plan-indicaciones.component.html Outdated
<plex-icon name="informacion"></plex-icon>
</plex-badge>
</ng-container>
</ng-container>

<div *ngIf="isToday() && permisosMapaCamasService.indicacionesEjecutar && permisosMapaCamasService.indicacionesEjecutar && !indicacionEventoSelected && !nuevaIndicacion && !indicacion.readonly"
Copy link
Contributor

Choose a reason for hiding this comment

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

Se podría comprimir esta condición, por: isToday() && permisosMapaCamasService.indicacionesEjecutar && !indicacionEventoSelected && !indicacion?.readonly

@negro89
Copy link
Contributor

negro89 commented Mar 14, 2024

Campe, el problema persiste cuando se elimina un borrador que se estaba editando.
image
image

@negro89 negro89 added changes requested Se solicitaron cambios and removed changes done labels Mar 14, 2024
@negro89 negro89 added Aprobado and removed changes requested Se solicitaron cambios labels Mar 15, 2024
@negro89 negro89 merged commit 8ef53e5 into master Mar 22, 2024
2 checks passed
@negro89 negro89 deleted the IN-583 branch March 22, 2024 18:07
liquid36 pushed a commit that referenced this pull request Mar 27, 2024
@semantic-release-bot
chore(release): 5.136.0 [skip ci]
ee2c3cf 
# [5.136.0](v5.135.1...v5.136.0) (2024-03-27)

### Bug Fixes

* **HUDS:** corrige error al visualizar solicitudes con adjuntos ([#2992](#2992)) ([eb27f1e](eb27f1e))

### Features

* **IN-583:** Eliminar indicacion en estado borrador ([#2954](#2954)) ([8ef53e5](8ef53e5))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Fabio-Ramirez Fabio-Ramirez Fabio-Ramirez left review comments

@AgosLizzi AgosLizzi AgosLizzi approved these changes

@negro89 negro89 negro89 approved these changes

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

Assignees

@ma7payne ma7payne

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.
7 participants
@MarianoCampetella @silviroa @negro89 @Fabio-Ramirez @AgosLizzi @ma7payne @MCele