WordPress.org

Make WordPress Core

Opened 8 years ago

Last modified 8 months ago

#6286 new enhancement

Proposed changes to "E-mail me whenever" Discussion Options

Reported by: markjaquith Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version: 2.5
Component: Comments Keywords: needs-patch
Focuses: ui, administration Cc:

Description

WRT the "E-mail me whenever" options on the Discussion options page:

http://img132.imageshack.us/img132/4215/picture1vf1.png

  1. For "a comment is helf for moderation," the "me" is ambiguous. It should specify that it means the blog admin e-mail address.
  2. For "anyone posts a comment," again, "me" is ambiguous. In this case "me" means the author of the post. The comment notification setting is personal, and therefore should be set in the profile options (where it can retain the "me"). Some authors may want e-mail notification, others might not.

Attachments (6)

6286.diff (2.5 KB) - added by ryan 6 years ago.
Moderation notification for post authors
individual notifications.diff (11.5 KB) - added by simonwheatley 6 years ago.
Individual notifications
individual notifications.2.diff (10.9 KB) - added by simonwheatley 6 years ago.
Individual notifications (removed stray error log calls)
6286.2.diff (7.5 KB) - added by wonderboymusic 3 years ago.
6286.3.diff (9.9 KB) - added by DrewAPicture 3 years ago.
Help text update, user-edit options indentation
6286.3.png (16.0 KB) - added by ocean90 3 years ago.

Download all attachments as: .zip

Change History (46)

#1 @Denis-de-Bernardy
7 years ago

  • Component changed from Administration to UI
  • Keywords needs-patch added
  • Milestone changed from 2.9 to Future Release

#2 @patulski
6 years ago

Currently using this solution to solve:
http://wordpress.org/support/topic/377874?replies=2

Would be nice if it were part of Core or at least if language was clarified.

@ryan
6 years ago

Moderation notification for post authors

#3 @ryan
6 years ago

Patch provides an interim solution of emailing the post author as well as the admin if the author has the proper caps. Ideally these will become individual options.

#4 @ryan
6 years ago

(In [16223]) Send moderation emails to post author. see #6286

#5 @nacin
6 years ago

  • Milestone changed from Future Release to 3.1

#4979 closed as a near-duplicate.

Moving to 3.1 as there's been a commit here.

@simonwheatley
6 years ago

Individual notifications

#6 @simonwheatley
6 years ago

Here's a suggested diff for individual notifications:

  • Move notifications from Options > Discussion to user profile/edit page
  • Individualised options are, "Email me whenever": "A comment is posted on one of my posts" "A comment on one of my posts is held for moderation" "Any comment is held for moderation"
  • New function wp_get_moderation_recipients
  • Removed old options from wherever I could see them (whitelists, schema)

N.B. No data migration currently in place to move options in an existing site to the new user_options, also no initial defaults in place for when new sites are set up

@simonwheatley
6 years ago

Individual notifications (removed stray error log calls)

#7 @westi
6 years ago

  • Cc westi added

This is looking like good progress towards a solution that we could implement in 3.2

Not sure if we just move this ticket there or create a new one as we have made some small changes in 3.1

#8 @ryan
6 years ago

  • Keywords 3.2-early added
  • Milestone changed from 3.1 to Future Release

Let's just move.

#9 @lonnylot
5 years ago

I would like to encourage this to be completed for 3.2 and if there is anything I can do to help this please let me know.

#10 @lonnylot
5 years ago

  • Cc lonnyk@… added

#11 @SergeyBiryukov
5 years ago

  • Keywords has-patch needs-refresh added; needs-patch removed

#14 @belg4mit
4 years ago

Please fix this. It might not be sexy to work on, but it's been an issue for a long time. There are numerous bug entries, and plenty of questions on the forums about how to get comment reply notifications sent to post authors.

#15 @wonderboymusic
3 years ago

  • Keywords needs-ui needs-testing added; needs-refresh removed

Refreshed the patch against trunk - needs testing, a strategy around options, and some UI love probs

#16 @helen
3 years ago

  • Component changed from UI to Administration
  • Keywords ui-focus added

@DrewAPicture
3 years ago

Help text update, user-edit options indentation

#17 @DrewAPicture
3 years ago

6286.3 adds a Personal Options help tab with a 'manage email notifications' bullet, also spaces the markup out in user-edit.

#18 @DrewAPicture
3 years ago

  • Cc DrewAPicture added

@ocean90
3 years ago

#19 follow-up: @ocean90
3 years ago

Not sure if the the second and the third checkbox should be checked at the same time.

Perhaps we can add a select box like on Discussion Settings where the user can select "any" or "one of my posts".

#20 @SergeyBiryukov
3 years ago

#7835 was marked as a duplicate.

#21 in reply to: ↑ 19 @DrewAPicture
3 years ago

Replying to ocean90:

Not sure if the the second and the third checkbox should be checked at the same time.

Perhaps we can add a select box like on Discussion Settings where the user can select "any" or "one of my posts".

What do you think about using a checkbox on the first option and either/or radio buttons for the second and third?

#22 follow-up: @azaozz
3 years ago

This all sounds good but still deals only with comments. Thinking that "E-mail me whenever..." should also include upgrade notifications.

I had this idea for a plugin for some time now (no time to sit down and make it): WordPress should be more "vocal" about upgrades by emailing the admin(s). The reason is that many users don't log in very often, sometimes for weeks or even months.

Thinking this would be quite handy on the Profile page:

E-mail me whenever:

  • .....
  • WordPress needs upgrading.
  • A plugin or a theme needs upgrading.

#23 in reply to: ↑ 22 ; follow-up: @SergeyBiryukov
3 years ago

Replying to DrewAPicture:

What do you think about using a checkbox on the first option and either/or radio buttons for the second and third?

Makes sense to me.

Replying to azaozz:

Thinking that "E-mail me whenever..." should also include upgrade notifications.

That would indeed be handy, but sounds like a separate enhancement to me.

#24 @SergeyBiryukov
3 years ago

#24383 was marked as a duplicate.

#25 @sillybean
3 years ago

  • Cc steph@… added

#26 in reply to: ↑ 23 @sillybean
3 years ago

Thinking that "E-mail me whenever..." should also include upgrade notifications.

That would indeed be handy, but sounds like a separate enhancement to me.

#10787 is the ticket for upgrade notifications.

#27 @sillybean
3 years ago

[25104] changes things a little; this patch will need an update.

#28 @jeremyfelt
2 years ago

  • Component changed from Administration to Comments
  • Focuses administration added

#29 @SergeyBiryukov
18 months ago

#30370 was marked as a duplicate.

#30 @SergeyBiryukov
9 months ago

#33572 was marked as a duplicate.

This ticket was mentioned in Slack in #design by helen. View the logs.


9 months ago

#32 @helen
9 months ago

  • Milestone changed from Future Release to 4.4

This whole screen is terrible. Let's figure it out as much as we can.

Related:

#22198
Realigning the Discussions Settings page
#23233
Radio Button instead of Checkboxes on Comment Moderation option?


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


8 months ago

#34 @dshanske
8 months ago

I would suggest we should change from Email to Notify. Email is not the only form of notification.

But re the issue of the settings, what if anyone who has the capability to moderate comments has the setting in their user profile to receive all moderation notifications?

#35 @wonderboymusic
8 months ago

  • Keywords needs-patch added; 3.2-early has-patch needs-ui needs-testing removed

The patch explodes - a ton has changed in 3 years

#36 follow-up: @johnbillion
8 months ago

I really like the Email Alerts plugin, which moves these notifications to a per-user setting.

#37 in reply to: ↑ 36 @helen
8 months ago

Replying to johnbillion:

I really like the Email Alerts plugin, which moves these notifications to a per-user setting.

Yeah I'm really thinking these should be user settings, although I think we probably have to retain the site settings that email the site admin since that's not inherently tied to a user account.

#38 @dshanske
8 months ago

Wasn't there a suggestion to hide them if the email address corresponds to an account?

#39 @DrewAPicture
8 months ago

  • Milestone changed from 4.4 to Future Release

Seems like we're still determining approach here, and screenshots would help. Moving off the 4.4 milestone pending a patch.

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


8 months ago

Note: See TracTickets for help on using tickets.