Make WordPress Core

Opened 7 weeks ago

Closed 4 weeks ago

#64007 closed enhancement (fixed)

Update Google Fonts collection for 6.9 release

Reported by: wildworks's profile wildworks Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 6.9 Priority: normal
Severity: normal Version:
Component: Editor Keywords: has-patch has-unit-tests needs-testing has-test-info
Focuses: Cc:

Description

Related past tickets:

The current Google font collection data is not up-to-date, with the latest additions to the list of available fonts and the latest versions of the existing ones.

This ticket is the central ticket for updating Google Fonts in the 6.9 release.

From my understanding, the tasks should be performed in the following order:

  1. Get a Google Fonts API key.
  2. Clone google-fonts-to-wordpress-collection repo locally, update fonts, and submit a PR with the latest fonts. Example of past PR: https://github.com/WordPress/google-fonts-to-wordpress-collection/pull/38
  3. After the PR is merged, submit a meta ticket to have the font list hosted. Example of a previous meta ticket: https://meta.trac.wordpress.org/ticket/7808
  4. After the new font is hosted, update the core code with the new URL. Example of past PR: https://github.com/WordPress/wordpress-develop/pull/6300

cc @mmaattiiaass @peterwilsoncc @mikachan @dd32

Change History (9)

#1 @wildworks
6 weeks ago

  • Type changed from defect (bug) to enhancement

#2 @wildworks
6 weeks ago

  • Milestone changed from Awaiting Review to 6.9

#3 @wildworks
6 weeks ago

  1. Get a Google Fonts API key.
  2. Clone google-fonts-to-wordpress-collection repo locally, update fonts, and submit a PR with the latest > fonts. Example of past PR: https://github.com/WordPress/google-fonts-to-wordpress-collection/pull/38

I've submitted a PR for the latest font: https://github.com/WordPress/google-fonts-to-wordpress-collection/pull/43

#4 @wildworks
5 weeks ago

  1. After the PR is merged, submit a meta ticket to have the font list hosted. Example of a previous meta ticket: https://meta.trac.wordpress.org/ticket/7808

Now that the PR has been merged, let's submit a meta ticket to have the fonts hosted.

#6 in reply to: ↑ 5 @dd32
5 weeks ago

Replying to wildworks:

Created a meta ticket: https://meta.trac.wordpress.org/ticket/8099

Resolved. CDN updated.

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


5 weeks ago
#7

  • Keywords has-patch has-unit-tests added

#8 @wildworks
5 weeks ago

  • Keywords needs-testing has-test-info added
  1. After the new font is hosted, update the core code with the new URL. Example of past PR: https://github.com/WordPress/wordpress-develop/pull/6300

The PR is ready. The testing instructions are also noted there.

https://github.com/WordPress/wordpress-develop/pull/10167

#9 @SergeyBiryukov
4 weeks ago

  • Owner set to SergeyBiryukov
  • Resolution set to fixed
  • Status changed from new to closed

In 60918:

Editor: Update Google Fonts API endpoint for WordPress 6.9.

Follow-up to [57558], [57686], [57871], [59295].

Props wildworks, dd32, mukesh27.
Fixes #64007.

Note: See TracTickets for help on using tickets.