devtools-core icon indicating copy to clipboard operation
devtools-core copied to clipboard

Hot Reloading regression

Open wldcordeiro opened this issue 8 years ago • 7 comments

Transferring issue from the debugger over thanks @irfanhudda for opening the original!

Running yarn start with "hotReloading": true gives error

ERROR in ./~/devtools-launchpad/src/utils/assert.js
Module not found: Error: Cannot resolve module 'react/lib/ReactUMDEntry/lib/ReactMount' in /home/mycomp/Projects/debugger.html/node_modules/devtools-launchpad/src/utils
 @ ./~/devtools-launchpad/src/utils/assert.js 1:316-347

After setting "hotReloading": false it gives no errors.

wldcordeiro avatar Feb 20 '17 03:02 wldcordeiro

I found that the issue is caused by us upgrading to React 15+ and using react-hot-loader 1.x. We need to upgrade to the 3.x version (currently beta).

Here's a link to a migration. https://github.com/gaearon/react-hot-loader/tree/next-docs/docs#migration-to-30

wldcordeiro avatar Feb 23 '17 08:02 wldcordeiro

It has been annoying me for a while that the hot reloading does not work. It would be great to address this issue as it would make everyone have a better experience fixing things in the frontend :-)

So i would like to have a go at this!

aklt avatar Jul 18 '17 19:07 aklt

I think it is better to wait until webpack 3 changes in PR#505 are landed and then fix HMR. As some API have changed drastically between wepack versions.

zaggy avatar Jul 18 '17 20:07 zaggy

Hi @zaggy, sounds reasonable, I'll keep an eye out on PR #505, thanks!

aklt avatar Jul 18 '17 22:07 aklt

I'll try again if I can get this running :)

aklt avatar Aug 11 '17 18:08 aklt

/claim

Anzumana avatar Dec 01 '17 14:12 Anzumana

@aklt would love to collab on this. msg me on slack or twitter if your up for it :)

Anzumana avatar Dec 01 '17 14:12 Anzumana