Skip to content

remove google fonts api call - #3700

Merged
akshay-gupta7 merged 8 commits into
mainfrom
akshay/remove-google-fonts
Dec 10, 2025
Merged

akshay-gupta7 merged 8 commits into
mainfrom
akshay/remove-google-fonts

Conversation

@akshay-gupta7

@akshay-gupta7 akshay-gupta7 commented Nov 24, 2025 •

Copy link
Copy Markdown
Contributor

Why does this PR exist?

Closes #3699

Removes usage of the Google Fonts API from the plugin, addressing security concerns.

What does this pull request do?

  • Removes all code that fetches fonts from Google Fonts API
  • Updates font loading logic to use only local or bundled assets
  • Cleans up any related configuration

Testing this change

  • Run the plugin and verify that no external calls are made to Google Fonts in devtools network tab
  • Ensure all fonts used by the plugin still render correctly
  • Confirm no errors or warnings appear related to missing fonts

Additional Notes (if any)

N/A

@changeset-bot

changeset-bot Bot commented Nov 24, 2025 •

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 6ec6572

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions

github-actions Bot commented Nov 24, 2025 •

Copy link
Copy Markdown
Contributor

Commit SHA:071e5a200b6c0e10030ccb64f359ef47c7a5e0fa
No changes to code coverage between the base branch and the head branch

@akshay-gupta7
akshay-gupta7 force-pushed the akshay/remove-google-fonts branch from 81756cf to d64ecb7 Compare November 25, 2025 15:34
@github-actions

github-actions Bot commented Nov 25, 2025 •

Copy link
Copy Markdown
Contributor

⤵️ 📦 ✨ The artifact was successfully created! Want to test it? Download it here 👀 🎁

@akshay-gupta7

Copy link
Copy Markdown
Contributor Author

@copilot can you investigate why is the cypress test failing in context to this pull request

Copilot AI commented Nov 25, 2025

Copy link
Copy Markdown
Contributor

@akshay-gupta7 I've opened a new pull request, #3701, to work on those changes. Once the pull request is ready, I'll request review from you.

@akshay-gupta7
akshay-gupta7 force-pushed the akshay/remove-google-fonts branch from c6b2d49 to 2d3f3b0 Compare November 25, 2025 17:58
@github-actions

github-actions Bot commented Dec 9, 2025 •

Copy link
Copy Markdown
Contributor

Commit SHA:a845d01ee29252b1ea52300ff3dcc9543020fa89
Current PR reduces the test coverage percentage by 1 for some tests

@akshay-gupta7 akshay-gupta7 self-assigned this Dec 9, 2025
@akshay-gupta7
akshay-gupta7 marked this pull request as ready for review December 9, 2025 19:18
@akshay-gupta7
akshay-gupta7 requested a review from six7 December 9, 2025 19:18
@akshay-gupta7
akshay-gupta7 merged commit c5b9154 into main Dec 10, 2025
8 of 12 checks passed
@akshay-gupta7
akshay-gupta7 deleted the akshay/remove-google-fonts branch December 10, 2025 13:03
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.

Check why we use Google Fonts

3 participants