Skip to content

IN - Plan de indicaciones: Permitir mas de una ejecución en una celda #3253

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
Open
MarianoCampetella wants to merge 2 commits into
base: master
Choose a base branch
from
Open

IN - Plan de indicaciones: Permitir mas de una ejecución en una celda #3253

MarianoCampetella wants to merge 2 commits into from

Conversation

@MarianoCampetella
Copy link
Contributor

@MarianoCampetella MarianoCampetella commented Oct 24, 2025 •
edited
Loading

Requerimiento

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

Se sumaron los siguientes cambios referenciados en las tareas:
https://proyectos.andes.gob.ar/browse/IN-672
https://proyectos.andes.gob.ar/browse/UIUX-446

Funcionalidad desarrollada

  1. Se permite cargar mas de un evento dentro de cada grilla horaria.
  2. Se pueden editar los eventos cargados dentro de las grillas.
  3. Si se agrega un nuevo evento dentro de una grilla que no tiene un circulo, solamente se podrá crear un evento con estado "REALIZADO"

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

@MarianoCampetella MarianoCampetella requested review from a team as code owners October 24, 2025 14:51
AgosLizzi
AgosLizzi requested changes Nov 6, 2025
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, te paso algunas correcciones:

  1. El botón editar queda cortado
image image
  1. Cuando estoy agregando un evento, no tengo la posibilidad de volver atras. Me parece que habría que considerar esta opción. Según lo planteado en este pr al agregar un evento directamente debo cerrar el sidebar. Sucede lo mismo al editar algún evento
image

Actualmente se esta trabajando en mejoras de visualización que tienen relación con este PR y tuve en cuenta los items que te acabo de mencionar

@AgosLizzi AgosLizzi added the changes requested Se solicitaron cambios label Nov 6, 2025
@MarianoCampetella MarianoCampetella added Consenso Requiere acordar algunos criterios and removed changes requested Se solicitaron cambios labels Nov 11, 2025
@AgosLizzi
Copy link
Contributor

AgosLizzi commented Dec 1, 2025

Campe, las modificaciones de este pr quedaron en esta tarea UX-UI-446. Se incluye la edición de un evento

@MarianoCampetella
Copy link
Contributor Author

MarianoCampetella commented Dec 10, 2025 •
edited
Loading

Todas las solicitudes de cambios se realizaron sobre este PR tomando como referencia los cambios desarrollados en la tarea https://proyectos.andes.gob.ar/browse/UIUX-446 junto con https://proyectos.andes.gob.ar/browse/IN-672

@MarianoCampetella MarianoCampetella added changes done and removed Consenso Requiere acordar algunos criterios labels Dec 10, 2025
@AgosLizzi AgosLizzi self-requested a review December 11, 2025 15:00
@MarianoCampetella MarianoCampetella force-pushed the IN-659 branch 2 times, most recently from 99afad4 to 736c2df Compare December 15, 2025 14:05
silviroa
silviroa requested changes Dec 17, 2025
View reviewed changes
Copy link
Contributor

@silviroa silviroa left a comment

Choose a reason for hiding this comment

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

Un detalle, cuando de agrega un evento... que ponga por defecto fecha y hora, por mas que luego tenga que cambiar el horario. Porque si tienen que agregar un solo evento tiene que hacer 2 clicks demás

@MarianoCampetella
Copy link
Contributor Author

MarianoCampetella commented Dec 18, 2025

Un detalle, cuando de agrega un evento... que ponga por defecto fecha y hora, por mas que luego tenga que cambiar el horario. Porque si tienen que agregar un solo evento tiene que hacer 2 clicks demás

Corregido, ahora cada vez que vaya a crear un nuevo evento se mostrará la hora correspondiente.

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

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

@silviroa silviroa Awaiting requested review from silviroa

Requested changes must be addressed to merge this pull request.

Assignees

No one assigned

Labels

changes done

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@MarianoCampetella @AgosLizzi @silviroa