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

Added babel-polyfill to support IE11 #611

Merged
merged 1 commit into from
Jun 21, 2016
Merged

Conversation

noisecapella
Copy link
Contributor

@noisecapella noisecapella commented Jun 21, 2016

What are the relevant tickets?

Fixes #609

What's this PR do?

Requires babel-polyfill for the dashboard.js required code. This is required for certain new pieces of the standard library like Object.assign

How should this be manually tested?

Download the IE11 virtual machine from IE11 and run the micromasters site from it.

Screenshots (if appropriate)

screenshot from 2016-06-21 12-33-11

What GIF best describes this PR or how it makes you feel?

@bdero bdero temporarily deployed to micromasters-ci-pr-611 June 21, 2016 16:35 Inactive
@gsidebo gsidebo self-assigned this Jun 21, 2016
@gsidebo
Copy link
Contributor

gsidebo commented Jun 21, 2016

👍

@noisecapella noisecapella merged commit 073eb49 into master Jun 21, 2016
@noisecapella noisecapella deleted the gs/babel-polyfill branch June 21, 2016 17:35
This was referenced Jul 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants