search

localgovdrupal / localgov_workflows

Default editorial workflow for LocalGov Drupal content.
GNU General Public License v2.0
0 stars 1 forks source link

[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 closed 4 months ago

finnlewis commented 4 months ago 
          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?

_Originally posted by @graham-web in https://github.com/localgovdrupal/localgov_workflows/issues/93#issuecomment-2097813402_