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

segmentio · Apr 20, 2023 · 2b71c10 · 2b71c10
1 parent a8b0fc8
commit 2b71c10
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 8 deletions.
diff --git a/.changeset/khaki-spoons-laugh.md b/.changeset/khaki-spoons-laugh.md
@@ -0,0 +1,5 @@
+---
+'@segment/analytics-next': patch
+---
+
+Fix 'Promise is undefined' issue with ie11 polyfill
diff --git a/packages/browser/src/browser/standalone.ts b/packages/browser/src/browser/standalone.ts
@@ -31,15 +31,14 @@ import {
 
 let ajsIdentifiedCSP = false
 
-const sendErrorMetrics = (() => {
+const sendErrorMetrics = (tags: string[]) => {
+  // this should not be instantied at the root, or it will break ie11.
   const metrics = new RemoteMetrics()
-  return (tags: string[]) => {
-    metrics.increment('analytics_js.invoke.error', [
-      ...tags,
-      `wk:${embeddedWriteKey()}`,
-    ])
-  }
-})()
+  metrics.increment('analytics_js.invoke.error', [
+    ...tags,
+    `wk:${embeddedWriteKey()}`,
+  ])
+}
 
 function onError(err?: unknown) {
   console.error('[analytics.js]', 'Failed to load Analytics.js', err)