You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
[Workflow notifications] Minor thought... shouldn't it also reset the last_email_run state when the "Enable email notifications" box has become ticked? Else the user would receive a flood of old notifications as soon as they [re-]enable e-mails!
#94
Closed
finnlewis opened this issue
May 7, 2024
· 0 comments
· Fixed by #96
Minor thought... shouldn't it also reset the `last_email_run` state when the "Enable email notifications" box has become ticked? Else the user would receive a flood of old notifications as soon as they [re-]enable e-mails!
Or perhaps simpler, for the same effect, just update state every time within hook_cron during the early-return?
Or perhaps simpler, for the same effect, just update state every time within
hook_cron
during the early-return?Originally posted by @graham-web in #93 (comment)
The text was updated successfully, but these errors were encountered: