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

4.3.12 breaks polyfills for IE11 #1099

Closed
joacim-boive opened this issue Nov 8, 2018 · 15 comments
Closed

4.3.12 breaks polyfills for IE11 #1099

joacim-boive opened this issue Nov 8, 2018 · 15 comments

Comments

@joacim-boive
Copy link

joacim-boive commented Nov 8, 2018

Description

Upgrading from 4.3.11 to 4.3.12 breaks polyfilling for IE11.

Expected behavior

It should continue to work. :)

Actual behavior

No error messages during build but polyfills stops working.
They seem to be in the code but not executing or something.

Environment

React Hot Loader version: 4.3.12
webpack 4.25.1
@babel/preset-env 7.1.5

Run these commands in the project folder and fill in their results:

  1. node -v: 10.13.0
  2. npm -v: 6.4.1

Then, specify:

  1. Operating system: macOS Mojave
  2. Browser and version: IE 11

Reproducible Demo

No repo at this time - sorry. :(

@skarjalainen
Copy link

This file:
https://github.com/gaearon/react-hot-loader/blob/master/src/proxy/createClassProxy.js
Uses string method that IE11 does not support:
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/startsWith

I see error in IE11 console:
SCRIPT438: Object doesn't support property or method 'startsWith'

@theKashey
Copy link
Collaborator

I'll release a fix tomorrow. It's to late for me to draft any release today.

@rgllm
Copy link

rgllm commented Nov 11, 2018

Hi @theKashey, any updates on this? Thanks.

@theKashey
Copy link
Collaborator

Fell free to upgrade to 4.4.0-1 (beta). It fixes this and another issues.

@rgllm
Copy link

rgllm commented Nov 12, 2018

Hi @theKashey,
just tested 4.4.0-1 (beta) and I confirm the issue is solved. However, it gives another error related to the Polyfill.
Console print.

@theKashey
Copy link
Collaborator

My bad, sorry

@rgllm
Copy link

rgllm commented Nov 13, 2018

Following on, @theKashey do you have any time expectations to solve this?

@theKashey
Copy link
Collaborator

🐢 tomorrow morning (GMT+10) 🐌
Sorry, was partially offline for two days.

@joacim-boive
Copy link
Author

Any updates? :)

@pfftdammitchris
Copy link

Great find! This definitely should be the reason why my app on IE is getting this error.

@theKashey
Copy link
Collaborator

Sorry for delay. I wanted to release this change with another one, but (and this is the nature of RHL) even having all the tests green - it does not work in real. Trying to find problem...

theKashey added a commit that referenced this issue Nov 20, 2018
@theKashey
Copy link
Collaborator

Fixed in "next", v4.5.1

@joacim-boive
Copy link
Author

Works great so far! :)

@theKashey
Copy link
Collaborator

:sweet:
So let me just collect some feedback before releasing a new version.

@theKashey
Copy link
Collaborator

4.6.0 is no longer beta.

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

No branches or pull requests

5 participants