Problem/Motivation
Currently, the commerce_abandoned_carts module automatically sends abandoned cart reminder emails based on the configured schedule. However, there is no built-in option to completely disable email sending.
This feature request proposes adding a configuration option to the module’s settings form that allows administrators to disable email notifications without disabling the module itself.
Steps to reproduce
- Enable commerce_abandoned_carts.
- Configure email settings.
- Emails are always sent based on the schedule, with no way to disable them from the UI.
Proposed resolution
- Add a new checkbox to the module’s configuration form at
/admin/commerce/config/abandoned_carts (or the relevant settings page).
- Label: Disable abandoned cart emails
- Description: "If checked, no abandoned cart emails will be sent, but tracking will continue."
- Default value: Unchecked (emails are enabled by default).
- Modify the email-sending logic to respect this setting.
Remaining tasks
- Implement the configuration setting in the admin UI.
- Modify the email dispatch logic to check this setting before sending emails.
- Update the module documentation, if necessary.
User interface changes
A new checkbox will be added to the module's configuration form.
API changes
No API changes are expected.
Data model changes
A new configuration setting will be added to store the email disable option.
Comments
Comment #2
denist3r commentedComment #4
denist3r commentedAdded patch for the 2.0.x branch
Comment #5
denist3r commentedComment #6
megachrizThanks for your contribution! It looks like a valuable addition to me.
Do you want to add test coverage for it too? This way we can ensure that no mails are being sent when the option is enabled.
I'm also thinking if it would be better UX to reverse the option label, so that it says 'Send abandoned cart emails' (and have it enabled by default). Because I believe that I read somewhere that enabling an option to disable a feature can be confusing.
Comment #7
megachrizComment #8
finex commentedI agree with @megachriz to reverse the option label.
Comment #9
denist3r commentedWorking on it!
Comment #10
denist3r commented