angular-redux / ng-redux

Angular bindings for Redux
MIT License
1.16k stars 177 forks source link

fix(build): Removed browser entry from package.json #160

Closed deini closed 7 years ago

deini commented 7 years ago

Fix for https://github.com/angular-redux/ng-redux/issues/156:

So there were 2 issues, first the replace plugin has to go before other plugins such as commonjs.

Also we had browser in package.json which Webpack will default to when resolving modules, so instead of resolving to lib/ng-redux.js (main) or es/ng-redux.js (module) it was resolving to dist/ng-redux.js which we build with env set to development https://github.com/angular-redux/ng-redux/blob/e1973f7d95059ec6f36259d972c9e69170f248ec/package.json#L20

FYI @maxlapides