Skip to content

rework dotorg_connectivity site health check - 2 - #337

Merged
joedolson merged 4 commits into
mainfrom
replace-site-health-connectivity-2
Dec 4, 2025
Merged

rework dotorg_connectivity site health check - 2#337
joedolson merged 4 commits into
mainfrom
replace-site-health-connectivity-2

Conversation

@joedolson

Copy link
Copy Markdown
Member

This is a rebuild of PR#30 after I made a complete hash of rebasing following a missed signoff.

The work here is by @ideag, props due to @circuitaxel.

Signed-off-by: Joe Dolson <design@joedolson.com>
@github-actions

Copy link
Copy Markdown
Contributor

@joedolson

joedolson commented Nov 23, 2025

Copy link
Copy Markdown
Member Author

Now that this is my own PR, of course, I can't review it... I did, however, resolve the PHPCS issues from the previous PR.

Signed-off-by: Joe Dolson <design@joedolson.com>
@joedolson

Copy link
Copy Markdown
Member Author

Linking to related issue: #9

…ivity.

Signed-off-by: Joe Dolson <design@joedolson.com>
Signed-off-by: Joe Dolson <design@joedolson.com>
@joedolson

Copy link
Copy Markdown
Member Author

I've tested the JS filtering, and it's working to filter this message correctly when there's an error, but wasn't filtering the WordPress connectivity fields on the Info tab.

That's filterable in PHP using the debug_information filter, so I added a simple filter to handle that case.

@joedolson
joedolson merged commit 9013140 into main Dec 4, 2025
46 checks passed
@joedolson
joedolson deleted the replace-site-health-connectivity-2 branch December 4, 2025 17:16
@costdev costdev added this to the 1.2 milestone Dec 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants