[wp-trac] [WordPress Trac] #43857: Show the comment / awaiting moderation message even without opt-in
WordPress Trac
noreply at wordpress.org
Mon Sep 3 12:15:31 UTC 2018
#43857: Show the comment / awaiting moderation message even without opt-in
-------------------------------------------------+-------------------------
Reporter: imath | Owner: javorszky
Type: defect (bug) | Status: assigned
Priority: normal | Milestone: 4.9.9
Component: Privacy | Version: 4.9.6
Severity: normal | Resolution:
Keywords: has-patch needs-testing needs- | Focuses:
refresh |
-------------------------------------------------+-------------------------
Changes (by javorszky):
* keywords: has-patch needs-testing => has-patch needs-testing needs-
refresh
Comment:
My username (which is my last name) is missing a 'z'. :)
Things I'm missing at the moment:
* refreshed patch - using the docs here:
https://make.wordpress.org/core/handbook/testing/patch/, applying the
patch via the grunt way fails to apply 2 of 3 hunks, so it doesn't seem
like I can test this in any meaningful way as of this moment
* as someone who's new to testing this part of the code, I need a few
steps to set up the opening state of things. What settings should
WordPress have, how to have a user submit a comment without consent, etc?
* then a way to see what this patch should be solving. For example: "If
(after common setup steps) user does X, without this patch, this happens.
With the patch, this other thing happens". Gifs and screenshots help.
Replying to [comment:47 robdxw]:
> @javorsky Is there anything I can do to help get this in 4.9.9? Unit
tests are all passing, and I've done some local testing which all seems
ok. But it's not clear from
[https://make.wordpress.org/core/handbook/testing/patch/] whether that's
sufficient, or if anything else is required.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/43857#comment:48>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list