WordPress.org

Make WordPress Core

Opened 20 months ago

Last modified 7 days ago

#43805 reviewing enhancement

Add "in reply to" in comment moderation email notification

Reported by: galbaras Owned by: SergeyBiryukov
Milestone: 5.4 Priority: normal
Severity: normal Version:
Component: Comments Keywords: has-patch needs-testing
Focuses: administration Cc:
PR Number:

Description (last modified by SergeyBiryukov)

The comment moderation notification email provides quick ways to moderate comments, which can save time and hassle.

Sadly, the "in reply to" information, which is available on the moderation page, is missing from these emails, so moderators lack some of the context of the comment.

As you can imagine, the meaning of a comment is vastly different when it relates to the post, as opposed to some stupid comment that someone made, and may make a difference to the moderation outcome.

Therefore, at least when reply threading is enabled, moderation emails should include the "in reply to" comment link and author name.

While we're at it, both the moderation page and email would benefit from the actual replied-to comment to be shown, in order to avoid the hassle of another page load to enable proper moderation.

This can be done in combination with #33627, where every change done on the moderation page is also reflected in the emails.

Attachments (2)

43805.diff (718 bytes) - added by danieltj 20 months ago.
Include post parent URL in notify mod email
43805.2.diff (1011 bytes) - added by andraganescu 7 days ago.
Refreshed patch 43805.diff

Download all attachments as: .zip

Change History (10)

#1 @danieltj
20 months ago

  • Keywords needs-patch added

I'm for adding an in reply to link with a link to the comment within the wp-admin, but I'm against any other information. Considering the fact you're moderating the reply, having the replied to comment itself doesn't seem necessary here.

@danieltj
20 months ago

Include post parent URL in notify mod email

#2 @danieltj
20 months ago

  • Focuses administration added
  • Keywords has-patch needs-testing added; needs-patch removed

I've added 43805.diff which includes the URL to the parent comment if one has been set. I was going to change the milestone to Future Release but let's see what people think. Haven't managed to test the patch yet, but if someone wants to give it a try and provide any feedback, please do.

#3 @SergeyBiryukov
20 months ago

  • Description modified (diff)

#5 @SergeyBiryukov
6 months ago

  • Milestone changed from Awaiting Review to 5.3
  • Owner set to SergeyBiryukov
  • Status changed from new to reviewing

#6 @davidbaumwald
3 months ago

  • Milestone changed from 5.3 to Future Release

This ticket hasn't received much attention this cycle. With the deadline for enhancements in 5.3 now passed with Beta 1, this is being moved to Future Release.

#7 @SergeyBiryukov
10 days ago

  • Milestone changed from Future Release to 5.4

@andraganescu
7 days ago

Refreshed patch 43805.diff

#8 @andraganescu
7 days ago

I refreshed the patch in 43805.2.diff because the existing one did not apply cleanly with grunt patch:43805.

Still needs testing.

Note: See TracTickets for help on using tickets.