Skip to content
This repository has been archived by the owner on Sep 11, 2018. It is now read-only.

Error on dev:nw #110

Closed
neverfox opened this issue Sep 29, 2015 · 3 comments
Closed

Error on dev:nw #110

neverfox opened this issue Sep 29, 2015 · 3 comments
Labels

Comments

@neverfox
Copy link
Contributor

I get the following redbox (Chrome Version 45.0.2454.99 (64-bit)) right off the bat when I run npm run dev:nw:

TypeError: Cannot read property 'location' of undefined
createDevToolsWindow
http://localhost:3000/app.5e06e61d60684546aa6e.js?5e06e61d60684546aa6e:2496:7
Root.renderDevTools
http://localhost:3000/app.5e06e61d60684546aa6e.js?5e06e61d60684546aa6e:4252:42
Root.proxiedMethod
http://localhost:3000/app.5e06e61d60684546aa6e.js?5e06e61d60684546aa6e:5348:31
Root.render
http://localhost:3000/app.5e06e61d60684546aa6e.js?5e06e61d60684546aa6e:4283:28
Root.proxiedMethod
http://localhost:3000/app.5e06e61d60684546aa6e.js?5e06e61d60684546aa6e:5348:31
Root.tryRender
http://localhost:3000/app.5e06e61d60684546aa6e.js?5e06e61d60684546aa6e:1151:32
ReactCompositeComponentMixin._renderValidatedComponentWithoutOwnerOrContext
http://localhost:3000/vendor.5e06e61d60684546aa6e.js?5e06e61d60684546aa6e:25834:35
ReactCompositeComponentMixin._renderValidatedComponent
http://localhost:3000/vendor.5e06e61d60684546aa6e.js?5e06e61d60684546aa6e:25854:33
wrapper
http://localhost:3000/vendor.5e06e61d60684546aa6e.js?5e06e61d60684546aa6e:2806:22
ReactCompositeComponentMixin.mountComponent
http://localhost:3000/vendor.5e06e61d60684546aa6e.js?5e06e61d60684546aa6e:25467:31
@dvdzkwsk dvdzkwsk added the bug label Sep 29, 2015
@vildehav
Copy link

If you allow pop-ups in chrome the problem goes away (Did for me anyway).

image

@dvdzkwsk
Copy link
Owner

@neverfox does what @vildehav said fix your issue?

@ptim
Copy link
Contributor

ptim commented Oct 1, 2015

Allowing popups fixes the issue for me, thanks!

ptim added a commit to ptim/react-redux-starter-kit that referenced this issue Oct 1, 2015
dvdzkwsk pushed a commit that referenced this issue Oct 2, 2015
reduxjs/redux-devtools#127
(temporarily install patched redux-devtools until 0.14 is officially released)

Add repo field to package.json to silence npm warning

Document the need to allow popups for `npm run dev:nw` #110

Use https:// rather than ssh:// for temporary redux-devtools patch
also, specify the only change necessary to fix console warnings
tomatau/redux-devtools@d77ef99

Update monkey-patched redux-devtools to include built files

Wait for the redux-devtools window to finish reloading before inserting our target node
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

4 participants