ReferenceError: window is not defined

[trevordmiller]

When using the setup recommended in react-transform-boilerplate by @gaearon, the .babelrc requires in redbox-react which causes an error because window doesn't exist in Node:

> mocha ./src/**/*.test.js --compilers js:babel/register

/Users/trevordmiller/projects/playground/nexus/node_modules/redbox-react/dist/redbox.js:504
            return /msie [6-9]\b/.test(window.navigator.userAgent.toLowerCase());
                                       ^
ReferenceError: window is not defined

It would be cool to have a environment check so that this error isn't thrown when the package is require or imported.

[trevordmiller]

Also, discussion around this issue has been happening in the react-transform channel in the reactiflux Slack team: https://reactiflux.slack.com/

[gaearon]

(Just FYI—while this is being fixed, you can easily use any other component for error reporting, including a custom one. It's configurable in .babelrc.)

[mathieumg]

It would be cool to have a environment check so that this error isn't thrown when the package is require or imported.

👍

you can easily use any other component for error reporting

@aulizko has done https://github.com/aulizko/delicate-error-reporter in the meantime :)

[trevordmiller]

@mathieumg Yeah, I switched to @aulizko's package and it has worked well running from both the server and client since it has an environment check. The unobtrusive errors are a plus as well :)

[gaearon]

Anybody wants to make a PR for this?

[aulizko]

@gaearon I'm working on PR fixing this right now ;)

[gaearon]

👍

[gaearon]

👍