[wp-trac] [WordPress Trac] #33717: Send Notification Email When a Comment is Approved From Moderation
WordPress Trac
noreply at wordpress.org
Sun Sep 20 11:11:24 UTC 2015
#33717: Send Notification Email When a Comment is Approved From Moderation
-----------------------------+-------------------------
Reporter: jeffr0 | Owner: swissspidy
Type: feature request | Status: assigned
Priority: normal | Milestone: 4.4
Component: Comments | Version:
Severity: normal | Resolution:
Keywords: has-patch | Focuses:
-----------------------------+-------------------------
Comment (by swissspidy):
Replying to [comment:32 obenland]:
> `Notify a comment author when his comment gets approved.` - Let's use a
gender-neutral `their`.
Done in the latest patch.
> We usually don't add an empty line between `@param` and `@return`
(unfortunately).
I know, but my IDE unfortunately always adds an empty line, even though I
configured all sniffs :/
> The meta key `_wp_notification_sent` seems a bit generic.
Renamed it to `_wp_commenter_notification_sent`.
> I also wonder if we should accept both a comment ID and a WP_Comment
object to be more in line with `wp_notify_postauthor()`.
Good idea. That way the function can also be hooked to other actions where
only an ID is passed. Added now.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/33717#comment:33>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list