Skip to content

Closes #5717: Disable all user module email notifications by default. - #5718

Open
joeparsons wants to merge 2 commits into
mainfrom
issue/5717-disable-user-module-notifications
Open

Closes #5717: Disable all user module email notifications by default.#5718
joeparsons wants to merge 2 commits into
mainfrom
issue/5717-disable-user-module-notifications

Conversation

@joeparsons

Copy link
Copy Markdown
Member

Description

Currently all of the default email notifications created by the core user module are all enabled except for these:

  • status_blocked: 'Account blocked'
  • status_canceled: 'Account canceled'

This PR disables all of the remaining email notification settings (by default):

  • cancel_confirm: 'Account cancellation confirmation'
  • password_reset: 'Notify user when password reset'
  • status_activated:'Notify user when account is activated'
  • register_admin_created: 'Welcome (new user created by administrator)'
  • register_no_approval_required: 'Welcome (no approval required)'
  • register_pending_approval: 'Welcome (awaiting approval)'

Release notes

Related issues

Closes #5717

How to test

Types of changes

Arizona Quickstart (install profile, custom modules, custom theme)

  • Patch release changes
    • Bug fix
    • Accessibility, performance, or security improvement
    • Critical institutional link or brand change
    • Adding experimental module
    • Update experimental module
  • Minor release changes
    • New feature
    • Breaking or visual change to existing behavior
    • Upgrade experimental module to stable
    • Enable existing module by default or database update
    • Non-critical brand change
    • New internal API or API improvement with backwards compatibility
    • Risky or disruptive cleanup to comply with coding standards
    • High-risk or disruptive change (requires upgrade path, risks regression, etc.)
  • Other or unknown
    • Other or unknown

Drupal core

  • Patch release changes
    • Security update
    • Patch level release (non-security bug-fix release)
    • Patch removal that's no longer necessary
  • Minor release changes
    • Major or minor level update
  • Other or unknown
    • Other or unknown

Drupal contrib projects

  • Patch release changes
    • Security update
    • Patch or minor level update
    • Add new module
    • Patch removal that's no longer necessary
  • Minor release changes
    • Major level update
  • Other or unknown
    • Other or unknown

Checklist

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • My change requires release notes.

@joeparsons joeparsons self-assigned this Jun 23, 2026
@joeparsons joeparsons added enhancement New feature or request needs discussion Further discussion required to determine requirements labels Jun 23, 2026
@joeparsons joeparsons linked an issue Jun 23, 2026 that may be closed by this pull request
@joeparsons joeparsons added the proposal Proposed change to how something works (usually larger or more fundamental than a feature request) label Jun 23, 2026
@joeparsons joeparsons moved this from Todo to In Progress in 3.5.0-alpha1 Jun 23, 2026
@joeparsons joeparsons changed the title Closes #5717: Disable user module email notifications. Closes #5717: Disable all user module email notifications by default. Jun 23, 2026
@joeparsons
joeparsons marked this pull request as ready for review June 23, 2026 22:24
@joeparsons
joeparsons requested review from a team as code owners June 23, 2026 22:24
@joeparsons joeparsons moved this from In Progress to Needs review in 3.5.0-alpha1 Jun 23, 2026
@tadean

tadean commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

This seems like it might have some unintended breaking changes for Quickstart sites which have user registration that actually use these emails potentially.

@joeparsons

Copy link
Copy Markdown
Member Author

This seems like it might have some unintended breaking changes for Quickstart sites which have user registration that actually use these emails potentially.

@tadean Do you mean because existing sites that actually rely on those emails could have them disabled when applying configuration updates?

Do you think including a release note about this change would be sufficient to minimize the impact of these changes on sites like that?

@joeparsons

joeparsons commented Jun 26, 2026

Copy link
Copy Markdown
Member Author

To do:

  • Determine whether there is a way to change these settings via the admin UI
  • Determine how to best handle changes to these settings for existing sites (DB update?)
  • Look into options for disabling emails only for CAS users

@joeparsons joeparsons moved this from Needs review to In Progress in 3.5.0-alpha1 Jun 26, 2026
@joshuasosa

Copy link
Copy Markdown
Contributor

I see UI checkbox options at /admin/config/people/accounts for:

  • Account activation: Notify user when account is activated
  • Account blocked: Notify user when account is blocked
  • Account canceled: Notify user when account is canceled

I think it would be neat to have UI-based options. A new page could be created within the Arizona Quickstart Settings area with those settings. A module could also potentially be used.

https://www.drupal.org/project/mailcontrol (D11 patch)
Adds checkboxes for:

  • Welcome (new user created by administrator)
  • Welcome (awaiting approval)
  • Welcome (no approval required)
  • Account cancellation confirmation
  • Password recovery

@tadean

tadean commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

I think it'd be reasonable to make this as a minor release change as long as there is a UI element where they can adjust the settings.

@joeparsons joeparsons moved this from Todo to In Progress in 3.6.0-alpha1 Jul 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request needs discussion Further discussion required to determine requirements proposal Proposed change to how something works (usually larger or more fundamental than a feature request) release notes

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

Disable user module email notifications by default

3 participants