-
Notifications
You must be signed in to change notification settings - Fork 18
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
Published Evidence site is getting 404 on subpages #6
Comments
I suspect (as a workaround) you could change your paths to be relative to the page they are on
i think evidence would then build those pages correctly, and the links would work. |
But agree that it currently works oddly with GitHub pages where the base of evidence is is not / |
@archiewood - Thanks very much for the pointer here. Replacing - [{revenue.city}](/{revenue.city})
+ [{revenue.city}](./{revenue.city}) FWIW, I also tested in my local machine where everything is hosted from root (`localhost), and there also the new relative paths work great. 🚀 This fixes the 404, but I notice that the breadcrumb parsing for the subpage seems to still be confused on these subpages:
Perhaps Evidence is trying to parse Is there are workaround for those, or will this require more broad support in Evidence for non-root publish locations? |
You've actually uncovered another v14 bug 🐛 |
This works fine:
https://meltanolabs.github.io/jaffle-shop-template/
But the pages that are generated render a 404 when clicked on or when the user hovers over the page title.
It looks like the pages are trying to get
https://meltanolabs.github.io/Philadelphia/
https://meltanolabs.github.io/Brooklyn/
instead of
https://meltanolabs.github.io/jaffle-shop-template/Philadelphia/
https://meltanolabs.github.io/jaffle-shop-template/Brooklyn/
The text was updated successfully, but these errors were encountered: