Remove Redux environment warning in production #4341
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
When running in production, there is a warning in the console:
That warning is present in both redux and ng-redux, but appears only because we're using the pre-built version of ng-redux, which was built with the development environment.
So... moving ng-redux to webpack so that we compile our own environment-dependnent version.
Also changed how we expose the environment - previously, for some reason we would take all the environment and pass it as a list of defaults to EnvironmentPlugin.
Now, we're only explicittly sharing
NODE_ENV
.Testing:
Check the browser console on the login screen.
(As far as I know, this is just a warning, no point in backporting unless somebody complains.)