Skip to content
This repository has been archived by the owner on Nov 28, 2022. It is now read-only.

🐛 we want to run babel through all of the js files to check for IE compatibility #221

Merged
merged 4 commits into from
Jul 8, 2019

Conversation

dok
Copy link
Contributor

@dok dok commented Jun 6, 2019

Fix is related to the ticket regarding .startsWith.

The fix is essentially to remove the babel exclusion for js files; This is allow babel to catch any files with compatibility issues with older browsers.

@mjcuva mjcuva requested a review from domharrington June 7, 2019 21:58
@dok
Copy link
Contributor Author

dok commented Jun 24, 2019

tested with IE11. It works!

@erunion I hear you're going to be taking over this repo and its deployment processes. How do you want me to notify you when something is ready to be merged and shipped?

I could tag as ready and assign you as a reviewer as a way

@erunion
Copy link
Member

erunion commented Jun 24, 2019

@dok Feel free to just @, or formally assign the PR to, me whenever you're ready.

@erunion erunion changed the title 🐛 we want to run babel through all of the js files to check for IE co… 🐛 we want to run babel through all of the js files to check for IE compatibility Jul 8, 2019
@erunion
Copy link
Member

erunion commented Jul 8, 2019

@dok Can you revert the react-jsonschema-form change, and resolve the conflicts with the two package-lock.json files?

This reverts commit b14bde3.
@dok
Copy link
Contributor Author

dok commented Jul 8, 2019

@erunion reverted

@erunion erunion merged commit eebb71d into master Jul 8, 2019
@erunion erunion deleted the fix/ie-compat branch July 8, 2019 20:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants