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 3 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
);
$emails[] = $arr;

// STIC-Custom 20240926 ART - “Run Email Reminder Notifications” does not run when deleting a person
Copy link
Collaborator

Choose a reason for hiding this comment

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

Indicar en el comentario que la tarea generaba un error bloqueante y en vez de persona indicaría invitado.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

hecho!!

Copy link
Collaborator

@ManuSinergiaCRM ManuSinergiaCRM Oct 7, 2024

Choose a reason for hiding this comment

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

Por claridad, propongo que en vez de añadir las dos líneas nuevas, modificar el comentario que ya existía:

// STIC-Custom 20240926 ART - Evitar la notificación a un invitado si este ya no existe en el CRM ya provocaba un error que impedía que finalizará correctamente la tarea “Run Email Reminder Notifications”
// #406

por el siguiente comentario o algo similar

// STIC-Custom 20240926 ART - #406
// Evitar el envío del recordatorio a cualquier invitado si este ya no existe en el CRM, ya que provoca un error que impide que la tarea “Run Email Reminder Notifications” finalice correctamente.

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)

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
2 participants