Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[feature-webpack-IE11-fix]: Bumping Webpack to fix dev server IE11 error. #273

Merged
merged 1 commit into from
Aug 2, 2018

Conversation

1Copenut
Copy link
Contributor

@1Copenut 1Copenut commented Aug 1, 2018

Purpose of the PR

The previous version of Webpack's hot module swap runtime was using the Array.prototype.includes method, and breaking in IE11. I've bumped the version of Webpack to use at least 4.16.1, with the package.json file for Webpack showing 4.16.3.

The HMR file can be viewed here: webpack/webpack@fcf634c#diff-f995afcfa4e47968fef81836a5da0bca

Changed

  • Webpack from 4.0.1 to 4.16.3

Fixed

  • Fixes a bug where the dev server will not render any output in IE11

Tested dev server rendering

  • IE11 - Windows 10
  • Edge - Windows 10
  • Chrome - Windows 10
  • Firefox - Windows 10
  • Safari - MacOS High Sierra
  • Chrome - MacOS High Sierra
  • Firefox - MacOS High Sierra

@pwolfert
Copy link
Collaborator

pwolfert commented Aug 1, 2018

Awesome, thank you for fixing this! LGTM 👍

@1Copenut 1Copenut merged commit 1dba444 into master Aug 2, 2018
@1Copenut 1Copenut deleted the feature/webpack-IE11-fix branch August 2, 2018 17:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants