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

Use polyfills to fix IE support #1935

Merged
merged 3 commits into from Mar 17, 2016
Merged

Use polyfills to fix IE support #1935

merged 3 commits into from Mar 17, 2016

Conversation

@edmundoa
Copy link
Member

@edmundoa edmundoa commented Mar 16, 2016

This PR fixes support for IE11, by polyfilling Promise. Here are the list of changes:

  • Use babel polyfill instead of our own Promise polyfill
  • Add a new chunk in webpack with polyfills
  • Load the polyfills JS file before plugins and the application

Fixes #1899

edmundoa added 3 commits Mar 15, 2016
We also need to load the polyfill chunk first when the Graylog server
generates the html template for the web interface.
@edmundoa edmundoa added this to the 2.0.0 milestone Mar 16, 2016
@bernd bernd self-assigned this Mar 17, 2016
bernd added a commit that referenced this pull request Mar 17, 2016
Use polyfills to fix IE support
@bernd bernd merged commit 0925f60 into master Mar 17, 2016
4 checks passed
4 checks passed
@garybot2
ci-server-integration Jenkins build graylog2-server-integration-pr 741 has succeeded
Details
@garybot2
ci-web-linter Jenkins build graylog-pr-linter-check 231 has succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@bernd bernd deleted the issue-1899 branch Mar 17, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

2 participants