Make WordPress Core

Opened 10 years ago

Last modified 4 weeks ago

#34293 assigned enhancement

Network Admin Email description doesn't really explain what it is.

Reported by: ipstenu's profile Ipstenu Owned by: johnjamesjacoby's profile johnjamesjacoby
Milestone: Future Release Priority: normal
Severity: normal Version:
Component: Networks and Sites Keywords: 2nd-opinion needs-copy-review
Focuses: multisite, ui-copy Cc:

Description

On /wp-admin/network/settings.php the field for Network Admin Email has this as the description:

This email address will receive notifications. Registration and support emails will also come from this address.

By contrast, the per-site has this:

This address is used for admin purposes, like new user notification.

I propose we change the Network Admin one to this:

This address is used for admin purposes, like site notifications. Registration and support emails will be sent from this address.

That makes for a little more parity, and explains more clearly that emails are sent FROM this address (which has been unclear to some).

The attached patch comes in two versions.

1) As I originally proposed
2) Without the 'and support' phrase since I have no idea what we are referring to with that one.

Attachments (2)

34293.1.diff (813 bytes) - added by Ipstenu 10 years ago.
With 'support'
34293.2.diff (801 bytes) - added by Ipstenu 10 years ago.
Without 'support'

Download all attachments as: .zip

Change History (28)

@Ipstenu
10 years ago

With 'support'

@Ipstenu
10 years ago

Without 'support'

This ticket was mentioned in Slack in #core-multisite by ipstenu. View the logs.


10 years ago

#2 @Ipstenu
10 years ago

Well this opened up a rats nest of issues.

This ticket will need to cover the following:

1) The verbiage is confusing
2) The purpose of the various email addresses are confusing

On single-site, the admin email receives notifications of new users and updates. The shadow email of wordpress@ is used to send emails to new users (password resets, etc).

On multisite, the admin email is used for both purposes for the network. It receives all emails concerning new users and sends all emails to users.

On multisite, the per-site admin email is somewhat less useful by core, but still used to send notifications to (not from).

In all cases, the language used does not fully describe the purpose of the email address being set.

We need to go over the following:

1) Single Single Admin Email

This address is used for admin purposes, like new user notification.

2) Network Admin Email

This email address will receive notifications. Registration and support emails will also come from this address.

3) Multisite per-site admin email

This address is used for admin purposes. If you change this we will send you an e-mail at your new address to confirm it. The new address will not become active until confirmed.

---

If we're going to provide examples for what 'admin purposes' are, we should do so across the board. Also we should say 'administrative' vs 'admin' since the former is more descriptive of what we're talking about.

1) Single Single Admin Email

This address will receive administrative emails, like updates and new user notifications.

2) Network Admin Email

This address will receive network administrative emails, like updates and new user notifications. Emails will be sent from this address, like to newly registered users and for password resets.

3) Multisite per-site admin email

This address will receive administrative emails. If you change this we will send you an e-mail at your new address to confirm it. The new address will not become active until confirmed.

That last one I'm not 100% sure on what emails it's actually going to get.

#4 @jeremyfelt
10 years ago

  • Milestone changed from Awaiting Review to Future Release

#5 @jeremyfelt
10 years ago

Also related, #27239.

This ticket was mentioned in Slack in #core-multisite by jjeaton. View the logs.


10 years ago

#7 @jeremyfelt
10 years ago

In 35575:

MS: Allow "Network Admin Email" to be a non-user during network setup.

The network admin email setting for a network is often used as a catch-all or notification email separate from the actual user ID set as the owner of the new network. If a non-user email address is set during network installation, we can defer to the current user as the actual network admin and apply the entered email as the address to which general notifications are sent and emails are sent from.

In the future, we'll want to update the messaging around "Network Admin Email" to reflect its reality. See #34293.

Props jjeaton.
Fixes #34065.

#9 @flixos90
9 years ago

Since we're getting rid of the term "Super Admin" and replace it with "Network Administrator" (see the ticket from above), the title "Network Admin Email" for this field gets somewhat confusing since this is not necessarily the email address of the network administrator (as there can be more than one) - for example there is a sentence in Core that would become Network administrator privileges cannot be removed because this user has the network admin email..

As @jeremyfelt stated in https://core.trac.wordpress.org/ticket/37593#comment:21, this email is rather a notification email, so I think at first glance "Network Notification Email" could be a good name, but there might still be a more appropriate one. It certainly makes sense to find a more suitable name, so let's discuss.

This ticket was mentioned in Slack in #core-multisite by flixos90. View the logs.


9 years ago

#11 @vizkr
9 years ago

Would it make more sense for this to be the "Network Owner Notification Email" or is that too nit picky?

This ticket was mentioned in Slack in #core by jeffpaul. View the logs.


9 years ago

This ticket was mentioned in Slack in #core-multisite by flixos90. View the logs.


9 years ago

This ticket was mentioned in Slack in #core by jeremyfelt. View the logs.


9 years ago

This ticket was mentioned in Slack in #core-multisite by flixos90. View the logs.


9 years ago

#16 @flixos90
9 years ago

  • Keywords 2nd-opinion added

#17 @johnjamesjacoby
9 years ago

I like "Network Notification Email" as a label, or even just "Network Email" if we want to keep things equally ambiguous. We could probably give sites the same treatment with a "Site Notification Email" too.

#18 @flixos90
9 years ago

@johnjamesjacoby I like the idea of using a simple term like "Network Email" and "Site Email", however in this case we need to make double-sure that the description of each of the three variants describes the usage of these email addresses precisely.

Regarding the restriction that prevents revoking super admin capabilities when that user has the same email address, anyone knows the history about that? My current thought is to get rid of this restriction as the contact / notification email should be independent of a user's email address in my opinion. This would also make the description easier to understand since it would (as far as my understanding goes) entirely be a notification address then.

#19 @flixos90
9 years ago

Related, I opened #39170 to remove the connection between the network email address and super admin users. This change would confirm that "Network Admin Email" is now an unsuitable label for the setting.

#20 follow-up: @johnjamesjacoby
9 years ago

anyone knows the history about that?

It was the only way to confirm "ownership" of a site or network (similar to what we talked about at WCUS and the site owner ticket.)

#21 in reply to: ↑ 20 @flixos90
9 years ago

Replying to johnjamesjacoby:

It was the only way to confirm "ownership" of a site or network (similar to what we talked about at WCUS and the site owner ticket.)

It isn't really functional though (inconsistent), and furthermore only works for the network admin. I think we're better off removing this connection and then investigate time thinking about an actual owner role (probably for both sites and networks) to build upon the WCUS discussion.

Let's continue discussion on that in #39170 though.

#22 follow-up: @johnjamesjacoby
4 months ago

  • Keywords i18n-change added
  • Milestone set to 6.9
  • Owner set to johnjamesjacoby
  • Status changed from new to assigned

The phrase "This address is used for admin purposes" has some redundancy now that it is adjacent to a label that says "Administration Email Address", which has me thinking it could be replaced completely with something more like one of @Ipstenu's original ideas of being more descriptive & intentional...

For the single-site site-level admin_email setting:

Receives system notifications from recovery mode, site health, automatic updates, password resets, and new user registrations.

And then if_multisite() we'd change it to...

Receives system notifications for this site, and used when sending password reset emails to users of this site.

...and then the network-level admin_email setting would say:

Receives system notifications from recovery mode, site health, automatic updates, password resets, new user registrations, and when new sites are created.

Thoughts?

#23 @sabernhardt
4 months ago

  • Focuses ui-copy added
  • Keywords needs-copy-review added; i18n-change removed

This ticket was mentioned in Slack in #core by welcher. View the logs.


2 months ago

#25 @jorbin
7 weeks ago

  • Milestone changed from 6.9 to Future Release

Punting to Future Release due to the impending 6.9 beta1 and lack of sustained progress during this cycle.

#26 in reply to: ↑ 22 @bridgetwillard
4 weeks ago

I think this is an excellent idea!

We build sites and hand them over, but not every client wants to be in charge.

I'd love for there to be a note about installing and configuring SMTP to speed up the delivery of site emails. That may be out of scope for this ticket.

Replying to johnjamesjacoby:

The phrase "This address is used for admin purposes" has some redundancy now that it is adjacent to a label that says "Administration Email Address", which has me thinking it could be replaced completely with something more like one of @Ipstenu's original ideas of being more descriptive & intentional...

For the single-site site-level admin_email setting:

Receives system notifications from recovery mode, site health, automatic updates, password resets, and new user registrations.

And then if_multisite() we'd change it to...

Receives system notifications for this site, and used when sending password reset emails to users of this site.

...and then the network-level admin_email setting would say:

Receives system notifications from recovery mode, site health, automatic updates, password resets, new user registrations, and when new sites are created.

Thoughts?

Note: See TracTickets for help on using tickets.