Skip to content

AST-5041 - Custom Fonts: .woff and .woff2 uploads blocked by WordPress file type restriction.#150

Open
yssambare12 wants to merge 2 commits intomasterfrom
woff-file-support
Open

AST-5041 - Custom Fonts: .woff and .woff2 uploads blocked by WordPress file type restriction.#150
yssambare12 wants to merge 2 commits intomasterfrom
woff-file-support

Conversation

@yssambare12
Copy link
Contributor

  • Custom Fonts: .woff and .woff2 uploads blocked by WordPress file type restriction.

Copilot AI review requested due to automatic review settings February 2, 2026 04:13
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds support for .woff and .woff2 font file uploads by updating WordPress MIME type handling. The changes also include support for .eot fonts and refactors the existing code for better maintainability.

Changes:

  • Added .woff and .woff2 MIME type handling to allow WordPress to accept these font formats
  • Added .eot font format support
  • Refactored the update_mime_types method to use a single $ext variable for consistency

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated 2 comments.

File Description
admin/dashboard/package-lock.json Dependency reorganization moving some packages to nested locations (likely from npm install)
admin/dashboard/includes/class-bsf-custom-fonts-menu.php Added MIME type handling for woff, woff2, and eot fonts; refactored code to use $ext variable
Files not reviewed (1)
  • admin/dashboard/package-lock.json: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@yssambare12 yssambare12 requested a review from a team February 2, 2026 04:18
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.

2 participants