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

fix: use cleanUrls in firebase config #12

Merged
merged 1 commit into from
Jul 25, 2024
Merged

fix: use cleanUrls in firebase config #12

merged 1 commit into from
Jul 25, 2024

Conversation

conorsch
Copy link
Contributor

Sets cleanUrls=true in the Firebase config [0], to fix how subpage navigation works on the hosted site. I'm not stoked that we need to perform this kind of URL rewriting at the hosting layer; I'd prefer that the build step emitted a properly structured static docroot.

Refs #8.

[0] https://firebase.google.com/docs/hosting/full-config#control_html_extensions

Sets `cleanUrls=true` in the Firebase config [0], to fix how subpage
navigation works on the hosted site. I'm not stoked that we need to
perform this kind of URL rewriting at the hosting layer; I'd prefer that
the build step emitted a properly structured static docroot.

Refs #8.

[0] https://firebase.google.com/docs/hosting/full-config#control_html_extensions
Copy link

Visit the preview URL for this PR (updated for commit 42455b9):

https://penumbra-guide--pr12-fix-sub-urls-a3fah7t9.web.app

(expires Thu, 01 Aug 2024 16:19:33 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: c64218fe717b75c9724a23946df883a1adf10faa

@conorsch conorsch merged commit 0fa8a31 into main Jul 25, 2024
2 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.

1 participant