Opened 13 years ago
Closed 2 years ago
#20007 closed enhancement (maybelater)
Add options & UI for admin registration and password reset notifications
Reported by: | Veraxus | Owned by: | |
---|---|---|---|
Milestone: | Priority: | normal | |
Severity: | normal | Version: | 3.4 |
Component: | Administration | Keywords: | has-patch needs-refresh changes-requested |
Focuses: | Cc: |
Description
This patch adds an single checkbox option to WordPress's Settings > Discussion page, allowing users to quickly toggle new user registration notifications/emails on or off.
While the notification function is currently pluggable, allowing non-technical users to easily toggle this feature themselves is both intuitively simple, and should prove generally advantageous to the community.
Attachments (4)
Change History (24)
#1
@
13 years ago
Only minor changes are needed to include this functionality. The following four files are affected:
/wp-admin/options-discussion.php
/wp-admin/options.php
/wp-includes/pluggable.php
/wp-admin/includes/schema.php
#2
@
13 years ago
Hi veraxus.
Could upload the patch in the correct format???
i couldnt track the changes
@
13 years ago
This patch includes everything from the previous wp-reg-notify-option-patch.diff patch but also adds an option for controlling whether or not admins want to be notified of password resets.
#6
@
12 years ago
- Version set to 3.4
Any chance this could make it into 3.4 beta 3? The changes are programmatically very minor and well tested (I've been using this tweak on dozens of production sites for months), but the usability improvements are substantial.
If needed, I can submit an updated patch based on the latest 3.4 trunk.
#13
@
9 years ago
- Focuses ui removed
- Keywords has-patch ux-feedback removed
- Milestone Awaiting Review deleted
- Resolution set to wontfix
- Status changed from new to closed
Given this is now the default way users get their passwords, this is no longer a good idea. Closing as wontfix
#14
@
9 years ago
- Keywords needs-refresh added
- Resolution wontfix deleted
- Status changed from closed to reopened
- Summary changed from Add option for new user registration notifications to Add options & UI for admin registration and password reset notifications
@chriscct7 - this targets admin emails only, not emails to users. WordPress still generates massive amounts of useless trash mail to admins as a result of user registrations and password resets, with no simple core UI to disable it. These are notifications that most admins and site owners do not need to receive. I've been using a pluginized version of this fix for years and, honestly, it should have been merged into core years ago.
Reopening and refreshing patch.
@
9 years ago
Adds options for admins to toggle admin notifications for new user and password resets.
#16
@
9 years ago
Patch updated for current 4.4 trunk. Allows admins to enable/disable admin notification emails for new user registrations and password resets. Again, this does not affect any emails or notifications to users - this is for admin notices only.
This feature is long overdue, let's get it done in 4.4!
This ticket was mentioned in Slack in #core by veraxus. View the logs.
9 years ago
#20
@
2 years ago
- Keywords needs-refresh changes-requested added; needs-testing needs-screenshots removed
- Resolution set to maybelater
- Status changed from reopened to closed
I'm going to close this out as a maybelater
.
While it's a valid request, it doesn't seem like a good change when evaluated against the project's philosophies. The lack of activity on this ticket in the last 10 years also is telling that it's not something that's in high demand.
Since this ticket was created, the code related to sending these emails has been updated to use an action hook instead of being called directly (see [34107] and [34251] from #33587). It's now much easier to disable these emails:
remove_action( 'after_password_reset', 'wp_password_change_notification' ); remove_action( 'register_new_user', 'wp_send_new_user_notifications' );
If this were to be reopened, the patch would need to be updated to reflect these changes.
Diff with minor changes to: /wp-admin/options-discussion.php, /wp-admin/options.php, /wp-includes/pluggable.php, and /wp-admin/includes/schema.php