Skip to content

fix: avoid CORS for docs font - #140

Merged
Tom Tang (qwerzl) merged 1 commit into
mainfrom
agent/fix-docs-font-cors
Aug 25, 2026
Merged

fix: avoid CORS for docs font#140
Tom Tang (qwerzl) merged 1 commit into
mainfrom
agent/fix-docs-font-cors

Conversation

@qwerzl

@qwerzl Tom Tang (qwerzl) commented Aug 25, 2026

Copy link
Copy Markdown
Member

Summary

  • Configure the custom font with its full https://photon.codes/docs/fonts/PolySansArabicVF.woff2 URL.
  • Prevent Mintlify from rewriting the font source to an inaccessible cross-origin S3 URL.

Validation

  • Local Mintlify preview preserves the Photon URL in font preloads and @font-face.
  • The Photon font URL returns 200 with font/woff2 and an Access-Control-Allow-Origin header.
  • Commit hooks passed lint, docs generation, and docs typechecking.

View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

Summary by CodeRabbit

  • Documentation
    • Updated heading and body font sources to use hosted documentation URLs for improved font loading reliability.

Copilot AI lite review requested due to automatic review settings August 25, 2026 07:11

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@coderabbitai

coderabbitai Bot commented Aug 25, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: fc16103a-dbbc-4141-b575-d6e237be6b45

📥 Commits

Reviewing files that changed from the base of the PR and between 9950537 and b303e55.

📒 Files selected for processing (1)
  • docs.base.json

Disabled knowledge base sources:

  • Linear integration is disabled

You can enable these sources in your CodeRabbit configuration.


📝 Walkthrough

Walkthrough

The documentation font declarations now load PolySansArabicVF from hosted Photon URLs instead of local paths.

Changes

Font Source Update

Layer / File(s) Summary
Hosted font URL declarations
docs.base.json
The heading and body PolySansArabicVF sources now use hosted Photon URLs.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Suggested reviewers: lcandy2

Poem

A rabbit checked the font path with care
Photon-hosted letters now fill the air
Headings and bodies hop in line
Each URL points to a source that shines
The docs burrow forward, neat and fine

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch agent/fix-docs-font-cors

Comment @coderabbitai help to get the list of available commands.

@qwerzl
Tom Tang (qwerzl) merged commit 4c11067 into main Aug 25, 2026
5 of 6 checks passed
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