-
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
Navigating from 404 page errors on production deployment #1838
Comments
I have the same error using a |
Attempted to reproduce with a the boilerplate example... Seems my personal deployment is causing the issue. Still digging into this for more info. |
Seems the most recent update has fixed this, @kbariotis try updating. |
@gtfargo hmm not really, still the same issue. |
I have same issue |
One of you want to take a crack at fixing this? |
@KyleAMathews Sure I will. Could you please guide me to the right direction in the codebase, since I haven't touched it before? |
In the if statement of this eventHandler in
|
The error is happening in https://github.com/gatsbyjs/gatsby/blob/master/packages/gatsby/cache-dir/production-app.js Follow the contributing instructions so you can develop against the checked out version of Gatsby and start adding console.logs to understand why there's the failure. |
@KyleAMathews @kbariotis @gtfargo I also just ran into this issue. Anything we can do to get the PR updated and merged soon? |
Im sorry @attfarhan didn't find the time to work on it :/ I need it too so I will try to check it again soon. |
fwiw this doesn't just affect links, it seems like none of the JS is being loaded in, (i.e. code called in componentDidMount is executed). @KyleAMathews is this a separate issue or is this resolved in your PR? |
This is still an issue. |
@KyleAMathews I can get this done if you don't mind 😃 This particularly happened when there's no history being pushed to |
That'd be great!
…On Sat, Oct 28, 2017, 6:36 PM Emmanuel Fallancy ***@***.***> wrote:
@KyleAMathews <https://github.com/kyleamathews> I can get this done if
you don't mind 😃 This particularly happened when there's no history
being pushed to window.___history
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1838 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAEVhx74TnYQtMCmOpE_ykrpSH4DJT3vks5sw9apgaJpZM4O5aAe>
.
|
in the meantime, any idea how to disable client-side on |
@revolunet do you mean as not using the |
no i mean, juste disable the client-side javascript on that page so links are clickable :) btw, i see PR #2667 is getting some love, so it may be better to focus on that real fix :) |
ah, right 😅 i see what you mean. |
Has anyone made meaningful progress on the fix? This is a user-impactful bug. @KyleAMathews Do you have a sense of where the issue lies? I'd be willing to take a try at the fix. |
@mikhail-softgrad this is fixed actually :-) @emmafallancy did several PRs a month or so ago so closing this now. |
Thanks, @KyleAMathews! For others in a similar situation, it looks like @emmafallancy's fix landed via #2667 and was released in |
It appears as though this issue was not yet fixed in v2. Can someone confirm this? |
When a user attempts to navigate from the 404 page on a production build, the console shows this error and the page never redirects:
or, if content for the desired destination is already loaded:
404.js
dependency versions:
The text was updated successfully, but these errors were encountered: