Make WordPress Core

Opened 3 months ago

Last modified 8 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 (6)

Screenshot.jpg (154.3 KB) - added by krupajnanda 8 weeks ago.
Screenshot 2024-05-10 at 10.51.27 AM.png (152.9 KB) - added by naeemhaque 12 days ago.
Test report #60719 by @naeemhaque
Screenshot at May 10 10-52-59.png (254.9 KB) - added by iamfarhan09 12 days ago.
Ticket #60719 @iamfarhan09
Test report #60719 by @ imasikur22.png (161.5 KB) - added by imasikur22 12 days ago.
Test report #60719 by @ imasikur22
Screenshot 2024-05-10 at 11.38.40 AM.png (149.5 KB) - added by tofajjal02 12 days ago.
Ticket #60719 @tofajjal02
Test report #60719.png (254.8 KB) - added by prottoysarkar 11 days ago.
Ticket #60719 @prottoysarkar

Download all attachments as: .zip

Change History (21)

#1 @swissspidy
3 months ago

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

Version 0, edited 3 months ago by swissspidy (next)

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


2 months 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.


8 weeks ago

#4 @krupajnanda
8 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.


8 weeks ago

@ironprogrammer commented on PR #6260:


6 weeks 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.

#7 @iamfarhan09
12 days ago

Bug Report

Description

According to this report, the patch indicated works as expected.

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

Environment

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

Actual Results

  1. ✅ I have tested the added changes with the given PR and they are working as expected.

Attachment Screenshot at May 10 10-52-59.png​ added

Last edited 12 days ago by iamfarhan09 (previous) (diff)

#8 @iamfarhan09
12 days ago

Last edited 12 days ago by iamfarhan09 (previous) (diff)

#9 @naeemhaque
12 days ago

Bug 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: 7.4.33
  • Server: nginx/1.23.2
  • Database: mysqli (Server: 8.0.36 / Client: mysqlnd 7.4.33)
  • Browser: Chrome 124.0.0.0
  • OS: macOS
  • Theme: Twenty Twenty-Four 1.0
  • MU Plugins: None activated
  • Plugins:
    • Test Reports 1.1.0

Steps to Reproduce

Actual Results

  1. ✅ The PR is working like expected result.
Last edited 12 days ago by naeemhaque (previous) (diff)

@naeemhaque
12 days ago

Test report #60719 by @naeemhaque

@iamfarhan09
12 days ago

Ticket #60719 @iamfarhan09

#10 @imasikur22
12 days ago

Bug Report

Description

According to this report, the patch indicated works as expected.
PR tested: https://github.com/WordPress/wordpress-develop/pull/6260

Environment

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

Expected Results

  1. ✅ The PR is working like expected result.

Additional Notes

  • The result is working fine but the status is showing "Loading..."

@imasikur22
12 days ago

Test report #60719 by @ imasikur22

#11 @tofajjal02
12 days ago

Bug Report

Description

According to this report, the patch indicated works as expected.

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

Environment

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

Actual Results

  1. ✅ The pr working is final as a expected result

#12 @sarkarripon
12 days ago

Bug Report

Description

According to this report, the patch indicated works as expected.
PR tested: https://github.com/WordPress/wordpress-develop/pull/6260

Environment

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

Actual Results

  1. ✅ PR worked fine from my end

Supplemental Artifacts

Add Inline: https://cln.sh/lZm38W0fsHWzcfNQPTf5

#13 @prottoysarkar
12 days ago

Bug Report

Description

According to this report, the patch described works as expected.

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

Environment

WordPress: 6.6-alpha-57778-src
PHP: 7.4.33
Server: nginx/1.25.1
Database: mysqli (Server: 8.0.37 / Client: mysqlnd 7.4.33)
Browser: Chrome 124.0.0.0
OS: macOS
Theme: Twenty Twenty-Four 1.0
MU Plugins: None activated
Plugins:
Test Reports 1.1.0

Actual Results

✅ I tested the changes with the given PR and they work as expected.

@prottoysarkar
11 days ago

Ticket #60719 @prottoysarkar

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


11 days ago

#15 @zunaid321
8 days ago

Bug 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.12
  • Server: nginx/1.25.3
  • Database: mysqli (Server: 8.0.36 / Client: mysqlnd 8.2.12)
  • Browser: Chrome 124.0.0.0
  • OS: Windows 10/11
  • Theme: Twenty Twenty-Four 1.0
  • MU Plugins: None activated
  • Plugins:
    • Test Reports 1.1.0

Expected Results

  1. ✅ PR worked fine but the status was stuck at "loading".

Supplemental Artifacts

Screenshot: https://i.ibb.co/WyTdskf/2024-05-13-14-13.png

Note: See TracTickets for help on using tickets.