WordPress.org

Make WordPress Core

Opened 5 years ago

Last modified 4 days ago

#10726 reopened enhancement

Admin notifications for more than 1 email

Reported by: novasource Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version: 2.8.4
Component: Administration Keywords:
Focuses: Cc:

Description

Please allow admin notifications to go to more than 1 email address. It could be as simple as allowing a comma-separated list.

Better yet, allow sending admin notifications to all people in the administrators role.

Change History (20)

comment:1 scribu5 years ago

  • Milestone changed from Unassigned to Future Release

comment:2 cnorris234 years ago

This has already been attempted before in #3969, and it was closed as won't fix. Mark Jaquith said it would be a big patch (and it would), and suggested that one set up an email address that forwards to all the admins (see comment 4). That being said, making the admin email filterable, should be a fairly easy/small patch, and would allow you to write a plugin to create this functionality.

comment:3 nacin4 years ago

  • Milestone Future Release deleted
  • Resolution set to wontfix
  • Status changed from new to closed

If anything we'd eventually refactor the admin email aspect in another direction and perhaps tie it to an account (or accounts). This is a source of confusion often. Closing as wontfix as had previously been done.

comment:4 westi3 years ago

  • Cc westi added
  • Milestone set to Future Release
  • Resolution wontfix deleted
  • Status changed from closed to reopened

I think we should re-consider this.

Ideally the option should go and we should send notifications to people based on caps/roles rather than a seconded hidden email address which might not be up-to-date

comment:5 follow-up: nacin3 years ago

This ticket proposed a comma-separated list. I closed this due to a few other tickets proposing the opposite direction (which is what you're suggesting be considered), such as an owner role or some other rework that moves away from the option and toward users. Problem is, now I can't find a ticket now.

comment:6 in reply to: ↑ 5 ; follow-up: RanYanivHartstein3 years ago

Replying to nacin:

This ticket proposed a comma-separated list. I closed this due to a few other tickets proposing the opposite direction (which is what you're suggesting be considered), such as an owner role or some other rework that moves away from the option and toward users. Problem is, now I can't find a ticket now.

I reopened this as a new ticket in #16481

comment:7 in reply to: ↑ 6 jane2 years ago

Replying to RanYanivHartstein:

I reopened this as a new ticket in #16481

Seems like 16481 is just asking to remove the email on General Settings, not to create a new 'owner' role.

comment:8 seancojr2 years ago

  • Cc seancojr added

comment:9 lightningspirit2 years ago

  • Cc lightningspirit@… added

comment:10 TheAdityaJain16 months ago

  • Cc mail@… added

comment:11 arippberger10 months ago

  • Cc arippberger added

comment:12 mau9 months ago

  • Cc ngomau@… added

comment:13 in reply to: ↑ description digabyte4 months ago

FOUR. YEARS. AGO.

For more than four years people have been asking for this... many, many people. How difficult can it be to allow a comma separated list for admin notification in the general settings?

Forcing a user to setup a dedicated admin e-mail account outside of WP with filters and forwarding is ridiculous when a simply comma separated list is all it takes to address the issue.

The first response to the original post states: "Mark Jaquith said it would be a big patch (and it would)" How in the world is a comma separated list a big patch? Perhaps the secondary suggestion of allowing all admins to receive admin notification is a big patch, but if so many people are requesting this functionality, isn't it just lazy to say "that's a big patch?"

I'm not necessarily advocating admin notification for all admins (though, that *does* seem to make perfect sense), but simply adding comma separated list functionality to admin notification should be addressed in the WP core; it should have been addressed years ago.

comment:14 supersuphot3 months ago

Wish to have this feature too

comment:15 follow-up: mordauk2 months ago

I do really like the idea of eventually moving away from a predefined email option, but what's the harm in allowing a comma delineated list of addresses? Seems the only thing we'd have to do is modify the way the option is sanitized so as to not strip out commans when multiple email addresses are detected.

comment:16 mordauk5 days ago

I will happily write a patch for this if we can consider it again.

comment:17 in reply to: ↑ 15 ; follow-up: rmccue4 days ago

Replying to mordauk:

Seems the only thing we'd have to do is modify the way the option is sanitized so as to not strip out commans when multiple email addresses are detected.

FWIW, this is pretty easy: array_filter( explode( ',', $email ), 'is_email' )

comment:18 in reply to: ↑ 17 mordauk4 days ago

Replying to rmccue:

Replying to mordauk:

Seems the only thing we'd have to do is modify the way the option is sanitized so as to not strip out commans when multiple email addresses are detected.

FWIW, this is pretty easy: array_filter( explode( ',', $email ), 'is_email' )

Yes. Why are we not allowing this? I fully understand providing a better UI for multiple emails is much more involved, but I haven't seen a reason why we can't support a comma separated list?

comment:19 follow-up: nacin4 days ago

Anyone using get_option( 'admin_email' ) and expecting a single email, versus a comma-separated list or an array, would cause breakage. Hence the desire to go bigger or not do anything at all.

comment:20 in reply to: ↑ 19 mordauk4 days ago

Replying to nacin:

Anyone using get_option( 'admin_email' ) and expecting a single email, versus a comma-separated list or an array, would cause breakage. Hence the desire to go bigger or not do anything at all.

Ah, hadn't thought of that.

Note: See TracTickets for help on using tickets.