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

Fix new schedules being a blank page in IE11 #366

Merged
merged 1 commit into from
Jan 31, 2020

Conversation

digitalcora
Copy link
Contributor

Asana Ticket: 🐞 New schedules page is blank in IE11

So as not to mess with our existing Webpack config at all, this creates a new config that explicitly compiles react-async using the same options. I've tested this using CBT and it allows IE11 to load the line diagram, though unfortunately the live data is still missing (and it's very unclear why).

@thecristen
Copy link
Collaborator

thecristen commented Jan 31, 2020

I'm looking into why this seems to break the javascript test suite -- so far it seems as though the few JS files that get used in some of our tests are not yet transpiled (but should already be transpiled in the test).

@codecov
Copy link

codecov bot commented Jan 31, 2020

Codecov Report

❗ No coverage uploaded for pull request base (master@0d86925). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #366   +/-   ##
=========================================
  Coverage          ?   93.96%           
=========================================
  Files             ?      356           
  Lines             ?     5946           
  Branches          ?        0           
=========================================
  Hits              ?     5587           
  Misses            ?      359           
  Partials          ?        0

@digitalcora digitalcora merged commit 0126913 into master Jan 31, 2020
@digitalcora digitalcora deleted the cfg-ie11-line-diagram branch January 31, 2020 20:38
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.

2 participants