Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Force the /fonts/ redirect #69

Closed
wants to merge 1 commit into from
Closed

Conversation

delucis
Copy link
Member

@delucis delucis commented Jul 11, 2024

Not sure if this is the cause of the inconsistencies a few people have seen with font loading, but might be worth a try.

Thinking:

  • This project includes a src/pages/[...rest].astro route, which in theory could try to handle /fonts/*.
  • Telling Netlify to force the redirect would in theory avoid that happening.

In practice, this doesn’t seem to be the case (and I can’t see it happening from logs), but if we hear that the problem is persisting this could be worth trying.

Copy link

netlify bot commented Jul 11, 2024

Deploy Preview for astro-new ready!

Name Link
🔨 Latest commit e0d04e5
🔍 Latest deploy log https://app.netlify.com/sites/astro-new/deploys/668fc3d62f608100080c85c6
😎 Deploy Preview https://deploy-preview-69--astro-new.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@delucis
Copy link
Member Author

delucis commented Jul 11, 2024

Got some feedback from @TheOtterlord and @florian-lefebvre that this issue seems to have resolved itself, so going to close this as surplus to requirements.

@delucis delucis closed this Jul 11, 2024
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.

1 participant