fix: polyfill Promise to avoid crashes in old browsers #1217
Conversation
Resolves #1213 This adds `Promise` polyfill before async entrypoint. This is "solution 2" in the linked issue. See there for more details.
There was a problem hiding this comment.
Thanks @ivan-aksamentov -- I've tested this and can find no faults, although I don't have access to IE so I'm trusting this fixes the issues you identified!
|
@emmahodcroft Could you please double-check https://dev.nextstrain.org in IE and pre-chromium Edge on your side? My understanding that this fix is now deployed there and it should work. |
|
Old edge: Looks good! I loaded main page & ncov/global, zoomed in and out a bit, & filtered. It seemed a bit slower than Chrome but that might just be my overloaded computer. But setting the colorings, for example, sometimes took a few seconds, and had a totally blank screen for half a second or so. That might be because it's |
Description of proposed changes
This adds
Promisepolyfill before async entrypoint.This is "solution 2" in the linked issue. See there for more details.
Related issue(s)
Fixes #1213
Testing
Tested in a some of the old browsers, as described in the issue
Thank you for contributing to Nextstrain!
Always glad