- Issue created by @scott_euser
- @scott_euser opened merge request.
- Issue was unassigned.
- Status changed to Needs review
over 1 year ago 4:02pm 26 July 2023 - π¬π§United Kingdom scott_euser
Merge request opened here https://git.drupalcode.org/project/simple_content_notifications/-/merge_...
Thanks! - πΊπΈUnited States aaronpinero
@scott_euser thank you for this suggestion. Since there was only one place where I was using the Drupal::messenger() service, I removed it.
I also adjusted the two main functions in the .module file (the hook_cron() and simple_content_notifications_prep()) to perform a check for the enabled status configuration value before doing anything else. This way nothing happens if that part of the module function is not enabled by the administrator.
To help make this more clear, I also adjusted the config forms so that the enable/disable option is the first item in each configuration form.
The changes are committed to the dev branch.
- Status changed to Fixed
over 1 year ago 5:10pm 31 July 2023 Automatically closed - issue fixed for 2 weeks with no activity.