[wp-trac] [WordPress Trac] #33717: Send Notification Email When a Comment is Approved From Moderation

WordPress Trac noreply at wordpress.org
Sun Jan 24 20:36:41 UTC 2021


#33717: Send Notification Email When a Comment is Approved From Moderation
--------------------------------------+--------------------------
 Reporter:  jeffr0                    |       Owner:  johnbillion
     Type:  feature request           |      Status:  reviewing
 Priority:  normal                    |   Milestone:  5.7
Component:  Comments                  |     Version:
 Severity:  normal                    |  Resolution:
 Keywords:  needs-dev-note has-patch  |     Focuses:
--------------------------------------+--------------------------

Comment (by johnbillion):

 This is looking really good. A few more points:

 * I've uploaded another patch, [attachment:"33717.13.diff"], mostly with
 formatting tweaks and improvements to the language used.
 * Unmoderated comments are only visible for one minute via the moderation
 hash ([https://github.com/WordPress/wordpress-
 develop/blob/0ceee0f871ac442f38e871b889eb6b77b611b487/src/wp-includes
 /class-wp.php#L408 ref]). I think this needs to be extended in order to
 allow users a longer window to opt in to the moderation notification and
 still be able to see their pending comment. If you're on a slow
 connection, you're using assistive technology, or you're a slow reader or
 slow decision maker, you can easily take longer than 60 seconds to submit
 this form. I can open a follow-up ticket for this.
 * I don't see much (any?) benefit to the messages displayed that say
 "{author} has opted in to receive a notification on comment’s approval".
 Does anyone feel strongly about these? I think they add quite a lot of
 visual noise but are of no concern to the user who is moderating comments.
 Thoughts?

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/33717#comment:75>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list