Skip to content
This repository has been archived by the owner on Jun 3, 2019. It is now read-only.

React Hot Loader error #165

Closed
carsonxyz opened this issue Nov 8, 2016 · 5 comments
Closed

React Hot Loader error #165

carsonxyz opened this issue Nov 8, 2016 · 5 comments

Comments

@carsonxyz
Copy link

It looks like the latest change moving react-hot-loader initialization to here has caused some issues.

I'm getting an error on first load:

React Hot Loader: It appears that "react-hot-loader/patch" did not run immediately before the app
started. Make sure that it runs before any other code. For example, if you use Webpack, you can
add "react-hot-loader/patch" as the very first item to the "entry" array in its config. Alternatively,
you can add require("react-hot-loader/patch") as the very first line in the application code, before
any other imports.
@ctrlplusb
Copy link
Owner

Woops! I accidentally checked a mess around session where I commented out the patch. It should exist in both places. I.e. the entry patch and the Babel plugin.

@ctrlplusb
Copy link
Owner

Sorted. Thanks for the report! :)

If you have upgraded to the latest version please let me know how things go for you. It's all very alpha the new code split module, but its already far more efficient that the previous one.

@carsonxyz
Copy link
Author

Already on the latest stuff on my personal project and working great so far.

I'm also about to open a PR updating the apollo branch to the latest.

@carsonxyz
Copy link
Author

Just noticed there is now a double inclusion of react-hot-loader patch at Line 134 and Line 380. Is that correct or is there an extra in there?

@ctrlplusb
Copy link
Owner

Yep, I believe they have seperate responsibilities in the RHL workflow.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants