Update package.json, react-dom added in peerDependencies section.

[sultan99]

Added missing react-dom package to peerDependencies.

It cause problem for some plugins like dynamic-cdn-webpack-plugin.

[netlify]

Deploy preview for react-redux-docs ready!

Built with commit 4792736

https://deploy-preview-1282--react-redux-docs.netlify.com

[timdorr]

We support React Native as well, which doesn't use react-dom as a renderer. So, we can't make this a peer or regular dependency.

[sultan99]

If you make it peer dependancy will it break react-native?

[sultan99]

@timdorr what about optional dependancy?

[timdorr]

If you make it peer dependency will it break react-native?

No, it will just warn upon install for every RN user.

An optional dep doesn't apply, as that is for non-production use cases.

[sultan99]

@timdorr I think optional dependency is right choice:

• npm-optional-dependency
• yarn-optional-dependency

and there is no deal with production or non-production use cases.

By refusing it you are depriving users of use cdn and other use cases.

[timdorr]

No, we're not. This library works just fine with a CDN. We publish a UMD bundle with every release and link to it from the package.json for Unpkg.

It just doesn't work for that particular Webpack plugin. You can still add a script tag manually to use it from a CDN.