[wp-trac] [WordPress Trac] #46957: Site Health: Make site health page access be filterable
WordPress Trac
noreply at wordpress.org
Sun Jun 9 04:04:27 UTC 2019
#46957: Site Health: Make site health page access be filterable
--------------------------------------------------+-----------------------
Reporter: Clorith | Owner: Clorith
Type: task (blessed) | Status: assigned
Priority: normal | Milestone: 5.2.2
Component: Administration | Version: 5.2
Severity: normal | Resolution:
Keywords: site-health has-patch has-unit-tests | Focuses:
--------------------------------------------------+-----------------------
Comment (by peterwilsoncc):
In [attachment:"46957.5.diff"]:
* Converts final four checks to `view_site_health_checks`
* Single site install: Ensures only administrators have above capability
* Multisite install: Ensures only super-admins have above capability
* Makes coding standards changes to capability test file required for
formatting test to pass
That final point adds a lot of white space changes, to view the patch
without white space [https://github.com/WordPress/wordpress-
develop/pull/67/files?w=1 view the pull request on GitHub with white space
changes suppressed].
[https://travis-ci.com/WordPress/wordpress-develop/builds/114851084
Witness the tests passing] on Travis CI.
**Next steps:**
I had to make rather more changes to the refresh than I expected to get
the tests passing. So could someone on this ticket/the site health project
please do the following within the next 24 hours so I can commit it:
* confirm the patch behaves as expected on a Multisite install
* confirm the patch behaves as expected on a Single site install
* add the `commit` keyword and ping me on the ticket if you give it the
thumbs up.
I have done the first two but code reviews and logic checks are some of my
favourite things.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/46957#comment:36>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list