-
Notifications
You must be signed in to change notification settings - Fork 10.3k
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
[1.0] building with prefix-link creates runtime err #1106
Comments
gatsby version : "1.0.0-alpha19-alpha.62172e76" |
If you're already using the |
Hey @nicholaswyoung, yea I'm using the I have two custom transformer plugins, but not sure that they are to blame cause everything works without prefix.
|
Do you know what component |
Here's the un uglified error.
|
Thanks, testing this. |
I added a linkPrefix to the gatsby-starter-blog and built it with Do you have a Do you have a |
Hey @KyleAMathews thanks for testing this, but it's not a build error, it's a browser runtime err which I think must have to do with when gatsby is building the routes. (I don't have any Route component in my app) I've reproduced with |
Hmm ok see it now. Going to work today on how we're loading code into the browser anyways so will tackle this as part of that. |
#1106 Also added an example site for path prefixing and fixed some bugs.
Just added an example site + some fixes for path prefixing in #1155 Will merge and release soon. |
* Change linkPrefix to pathPrefix to make link/path usage consistent fixes #1106 Also added an example site for path prefixing and fixed some bugs. * Update tests * More bug fixing * Remove console.log * handle pathnames with hashes * Force asynchronicity
Thanks that fixed this 👍 |
Pretty strange, when building with the prefix flag this runtime error
Running in dev mode or without the prefix flag I don't see it. Wondering if anyone else has encountered this or has ideas on the root cause?
The text was updated successfully, but these errors were encountered: