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.

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

Hotfix - General - Misma sensibilidad en minutos entre actualización masiva y la vista de Edición #79

Merged
merged 2 commits into from
Feb 19, 2024

Conversation

PaulaaSTIC
Copy link
Collaborator

@PaulaaSTIC PaulaaSTIC commented Jan 24, 2024

Descripción

Corrige el problema en el que los campos de fecha y hora en el formulario de actualización masiva no mostraban la misma sensibilidad de minutos que en la vista de edición.

Solución implementada:

La solución extiende los cambios realizados en los antiguos PRs 708, 949 y 1275.

Se modificó el archivo include/MassUpdate.php para que se llame al archivo custom/include/SugarFields/Fields/Datetimecombo/Datetimecombo.js. Siguiendo la lógica de este archivo, se utiliza la cadena de texto NOW en la propiedad display_default para indicar si se debe mostrar el intervalo minuto a minuto en campos de tipo DateTime.

Pruebas

  1. Acceder a Estudio / Sesiones / Campos / y editar el campo Fecha de inicio. Guardar indicando el valor NOW en la propiedad Valor por defecto. Aunque al entrar se indique NOW, guardar igualmente para que el valor por defecto cambie de now&09:00am a now.
  2. Crear una o dos sesiones y comprobar que la sensibilidad en minutos de la Fecha de Inicio es de un minuto.
  3. Seleccionar una o varias sesiones y pulsar en Actualización masiva.
  4. Comprobar que la sensibilidad en minutos del campo Fecha de Inicio es de un minuto.
  5. Ir al módulo de Inscripciones y comprobar que la sensibilidad para el campo Fecha de inscripción no es de un minuto.

@PaulaaSTIC PaulaaSTIC closed this Jan 25, 2024
Copy link

github-actions bot commented Jan 25, 2024

Actions executed at: 2024-02-19 16:44:55.

@PaulaaSTIC PaulaaSTIC reopened this Jan 25, 2024
@PaulaaSTIC PaulaaSTIC closed this Jan 26, 2024
@PaulaaSTIC PaulaaSTIC reopened this Jan 26, 2024
@PaulaaSTIC PaulaaSTIC closed this Jan 29, 2024
@PaulaaSTIC PaulaaSTIC reopened this Jan 29, 2024
@AlbertoSTIC AlbertoSTIC requested review from ManuSinergiaCRM and ainaraRT and removed request for AlbertoSTIC January 29, 2024 22:38
@AlbertoSTIC AlbertoSTIC removed their assignment Jan 29, 2024
Copy link
Collaborator

@ainaraRT ainaraRT left a comment

Choose a reason for hiding this comment

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

(A)Probado

Copy link
Collaborator

@ManuSinergiaCRM ManuSinergiaCRM left a comment

Choose a reason for hiding this comment

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

(A)probado

Comprobado que también funciona con el valor del campo Fecha de inicio por defecto: "now&09:00am"

@AlbertoSTIC AlbertoSTIC merged commit a476343 into develop Feb 19, 2024
3 of 7 checks passed
@AlbertoSTIC AlbertoSTIC deleted the bug/datetimeSensitivityInMassUpdate branch February 19, 2024 16:44
@PaulaaSTIC
Copy link
Collaborator Author

PR en SA: salesagility#10368

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working General not urgent
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Incidencia - General - Actualización masiva no tiene misma sensibilidad en minutos que la vista de Edición
4 participants