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

CIT - guardar motivo suspension de agenda #2733

Merged
merged 1 commit into from
Dec 6, 2022
Merged

CIT - guardar motivo suspension de agenda #2733

merged 1 commit into from
Dec 6, 2022

Conversation

MarianoCampetella
Copy link
Contributor

Requerimiento

https://proyectos.andes.gob.ar/browse/CIT-185

Funcionalidad desarrollada

  1. Se agrega el nombre del motivo de suspension a la constante patch.
  2. Se hizo un pequeño refactor para corregir las funcionalidades que se solapaban en el sidebar las cuales eran editar y suspender agenda.
  3. El select del motivo de suspencion se hizo un campo requerido ya que es necesario almacenarlo y evitar errores.

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 August 9, 2022 16:16
@martinebucarey
Copy link
Contributor

USUARIO: mcampetella
BUILD NUMBER: 7376
CYPRESS RUN: 6122
TEST START: 2022-08-09T16:40:03.589Z
TOTAL: 387
SUCCESS: 375
FAIL: 11
SKIPPED: 1

@martinebucarey
Copy link
Contributor

USUARIO: mcampetella
BUILD NUMBER: 7377
CYPRESS RUN: 6123
TEST START: 2022-08-09T17:42:58.471Z
TOTAL: 406
SUCCESS: 405
FAIL: 0
SKIPPED: 1

@martinebucarey martinebucarey added test ok Los test estan ok and removed test fail labels Aug 9, 2022
martinebucarey
martinebucarey requested changes Aug 11, 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.

Dos cosas visuales:
1- cuando borras el select aparece 2 veces el seleccionar el motivo, me parece que poniendose en rojo el select y que aparezca el placeholder esta bien
imagen
2- Estaría bueno que en el detalle de la agenda, si esta esta suspendida se muestre el motivo para dar la información al usuario.
imagen

@MarianoCampetella
Copy link
Contributor Author

Dos cosas visuales: 1- cuando borras el select aparece 2 veces el seleccionar el motivo, me parece que poniendose en rojo el select y que aparezca el placeholder esta bien imagen 2- Estaría bueno que en el detalle de la agenda, si esta esta suspendida se muestre el motivo para dar la información al usuario. imagen

Corregido!!

AgosLizzi
AgosLizzi requested changes Aug 18, 2022
View reviewed changes
Copy link
Contributor

@AgosLizzi AgosLizzi left a comment

Choose a reason for hiding this comment

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

Campe, cuando suspendo una agenda y en el motivo pongo "edilicia", luego sobre el plex help el motivo me muestra "profesional". Lo mismo sucede cuando en el motivo selecciono "organización". Te dejo captura con algunos errores sobre la consola

https://www.loom.com/share/d14ac3f8f998422daf8a2a261ebd4e54

@MarianoCampetella
Copy link
Contributor Author

Campe, cuando suspendo una agenda y en el motivo pongo "edilicia", luego sobre el plex help el motivo me muestra "profesional". Lo mismo sucede cuando en el motivo selecciono "organización". Te dejo captura con algunos errores sobre la consola

https://www.loom.com/share/d14ac3f8f998422daf8a2a261ebd4e54

Corregido!

@MCele MCele merged commit 6f983cc into master Dec 6, 2022
@MCele MCele deleted the CIT-185 branch December 6, 2022 15:25
liquid36 pushed a commit that referenced this pull request Dec 7, 2022
@semantic-release-bot
chore(release): 5.111.0 [skip ci]
b22e59e 
# [5.111.0](v5.110.0...v5.111.0) (2022-12-07)

### Features

* **CIT-185:** guardar motivo suspension de agenda ([#2733](#2733)) ([6f983cc](6f983cc))
* **IN-499:** Registros de estado dentro de una cama ([#2768](#2768)) ([6864788](6864788))
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

@martinebucarey martinebucarey martinebucarey approved these changes

@MCele MCele MCele approved these changes

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

Assignees

@negro89 negro89

@AgosLizzi AgosLizzi

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

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