Make WordPress Core

Opened 6 weeks ago

Last modified 4 days ago

#60719 new enhancement

Add fonts upload location to Site Health report

Reported by: swissspidy's profile swissspidy Owned by:
Milestone: 6.6 Priority: normal
Severity: normal Version:
Component: Site Health Keywords: good-first-bug has-patch has-screenshots has-testing-info
Focuses: Cc:

Description

Just like Site Health lists the paths to the plugins/themes/uploads directories, we should list the path to the fonts directory (and its size) there as well.

Attachments (1)

Screenshot.jpg (154.3 KB) - added by krupajnanda 3 weeks ago.

Download all attachments as: .zip

Change History (7)

#1 @swissspidy
6 weeks ago

There could also be a general check whether wp-content itself is writable or not.

Edit: this already exists!

Last edited 6 weeks ago by swissspidy (previous) (diff)

This ticket was mentioned in PR #6260 on WordPress/wordpress-develop by @andr3ribeiro.


5 weeks ago
#2

  • Keywords has-patch added; needs-patch removed

Added the font directory in the array for site health table iteration

This ticket was mentioned in Slack in #core-test by krupajnanda. View the logs.


3 weeks ago

#4 @krupajnanda
3 weeks ago

  • Keywords has-screenshots has-testing-info added

I tested changes with mentioned GH PR and it shows the font dir path and size to Site Health.

Test Report

Description

This report validates whether the indicated patch works as expected.

PR tested: https://github.com/WordPress/wordpress-develop/pull/6260

Environment

  • WordPress: 6.6-alpha-57778-src
  • PHP: 8.2.15
  • Server: nginx/1.25.3
  • Database: mysqli (Server: 8.0.36 / Client: mysqlnd 8.2.15)
  • Browser: Chrome 123.0.0.0
  • OS: macOS
  • Theme: Twenty Twenty-Four 1.0
  • MU Plugins: None activated
  • Plugins:
    • Test Reports 1.1.0

Actual Results

  1. ✅ Added changes are working as expected with given PR.

Additional Notes

  • Any additional details worth mentioning.

Supplemental Artifacts

Add as Attachment

This ticket was mentioned in Slack in #core by andr3ribeiro. View the logs.


3 weeks ago

@ironprogrammer commented on PR #6260:


4 days ago
#6

Thanks for the PR, @andr3ribeiro! Perhaps a good addition to this would be to include the fonts dir "is writable" check found on the Filesystem Permissions tab. This work is something we could probably get merged early this cycle.

Note: See TracTickets for help on using tickets.