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 - Reuniones - "Run Email Reminder Notifications" no se ejecuta al eliminar una persona #406

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

ainaraRT
Copy link
Collaborator

Desarrollo del issue

Se ha comprobado que al crear una reunión con recordatorios, tanto emergente como por correo, y se elimina una persona/usuario antes de que se ejecute la tarea programada "Run Email Reminder Notifications", falla la ejecución, haciendo que se quede estancada la tarea y desde ese momento, no se enviará ningún recordatorio.
Además, si lleva mucho tiempo estancada la tarea, las reuniones con fechas pasadas, se envían.

Solución implementada

  1. Se ha añadido una condición de que si lleva mucho tiempo la tarea estancada y ya ha pasado el recordatorio (fecha de inicio de la reunión, ejemplo: 16/06/2022 9:00), no se mande ningún recordatorio a los asistentes de la reunión pasada.
  2. Se ha añadido una condición de que si la persona queda eliminada, no se intente mandar un correo y continúe la ejecución dicha tarea.

Pruebas

Persona eliminada

  1. Configurar correctamente la cuenta de correo.
  2. Crear una reunión, añadiendo un recordatorio de correo a los asistentes.
  3. Guardar y enviar invitaciones.
  4. Eliminar la persona/usuario antes de ejecutar la tarea programada "Run Email Reminder Notifications".
  5. Ver en la reunión que aparece la persona como "UNKNOWN".
  6. Ejecutar la tarea "Run Email Reminder Notifications".
  7. Comprobar que llega el correo electrónico del recordatorio a los invitados, menos a la persona eliminada, y al creador de la reunión.
  8. Además, comprobar que la tarea ha sido ejecutada correctamente.

Reunión pasada

  1. Configurar correctamente la cuenta de correo.
  2. Crear una reunión pasada, añadiendo un recordatorio de correo a los asistentes.
  3. Guardar y enviar invitaciones.
  4. Ejecutar la tarea "Run Email Reminder Notifications".
  5. Comprobar que no llega el correo electrónico del recordatorio, a ninguno de los invitados ni el creador de la reunión.
  6. Además, comprobar que la tarea ha sido ejecutada correctamente.

@ainaraRT ainaraRT added bug Something isn't working Actividades Módulos de Llamadas, Reuniones, Tareas, Notas, Calendario... General labels Sep 26, 2024
@ainaraRT ainaraRT self-assigned this Sep 26, 2024
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.

Ver comentarios de la revisión

modules/Reminders/Reminder.php Outdated Show resolved Hide resolved
modules/Reminders/Reminder.php Outdated Show resolved Hide resolved
modules/Reminders/Reminder.php Show resolved Hide resolved
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.

Ver el comentario: #406 (comment)

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

Copy link
Collaborator

@jordiSTIC jordiSTIC 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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Actividades Módulos de Llamadas, Reuniones, Tareas, Notas, Calendario... bug Something isn't working General
Projects
None yet
3 participants