-
-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
Value is Undefined in SSR of React-Redux v6 #1107
Comments
Can you put together a project that reproduces the issue? |
Hi @markerikson, I'm still able to reproduce it on this project, https://github.com/sinwailam193/boilerplate. |
Ok I figure out the issue, it deals with the fact that it is using the server side rendering is using https://github.com/ctrlplusb/react-tree-walker, therefore it is causing this error. |
@sinwailam193: What did you do to resolve this? I'm having the same problem with SSR. |
Hey @DBosley, were you also using the https://github.com/ctrlplusb/react-universally boilerplate? If you are, I would say just don't use react-async-bootstrapper, it's what is causing this bug in the first place. |
There is an issue that I found in regarding to React Redux v6 on SSR, it might be related to the new Context.Consumer API.
I am currently getting an error in the terminal that states
However when I turn off SSR, the app works fine with no issue. So this issue is only on the server side rendering. I also dug into the new version 6 code and when I console.log Context that is being passed into the Connect class of connectAdvanced it logs
which contains the store and store state, however the issue is when it is being consumed by the Consumer
value is being undefined in this.renderWrappedComponent.
The expected behavior is that value should contain store and storeState from the store in Consumer on the server side, similar to the behavior of client side.
The version of redux is 4.0.1 and react-redux 6.0.0, Mac OS. This was not an issue in the previous version of react-redux 5.1.1 since it was using the old context API.
The text was updated successfully, but these errors were encountered: