Skip to content

Fix 'Promise is undefined' error that breaks ie11 auto-polyfill#842

Merged
silesky merged 1 commit intomasterfrom
detect-ie11
Apr 20, 2023
Merged

Fix 'Promise is undefined' error that breaks ie11 auto-polyfill#842
silesky merged 1 commit intomasterfrom
detect-ie11

Conversation

@silesky
Copy link
Copy Markdown
Contributor

@silesky silesky commented Apr 20, 2023

I accidentally introduced an ie11 regression when adding some CSP logic. Who knew eagerly instantiating RemoteMetrics would be the problem?

PS: Another data point for why we want webdriver tests that run browsers via a device farm (like I set up with action-destinations.)

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Apr 20, 2023

🦋 Changeset detected

Latest commit: 60b19d2

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@segment/analytics-next Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@silesky silesky changed the title fix Promise is undefined regression with ie11 polyfill fix 'Promise is undefined' regression that breaks ie11 auto-polyfill Apr 20, 2023
@silesky silesky changed the title fix 'Promise is undefined' regression that breaks ie11 auto-polyfill Fix 'Promise is undefined' error that breaks ie11 auto-polyfill Apr 20, 2023
@silesky silesky requested a review from pooyaj April 20, 2023 00:40
@silesky silesky merged commit 2b71c10 into master Apr 20, 2023
@silesky silesky deleted the detect-ie11 branch April 20, 2023 17:48
@github-actions github-actions bot mentioned this pull request Apr 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants