docs(js): Fix default integration examples

getsentry · Oct 16, 2024 · b52b122 · b52b122
1 parent 01c55e8
commit b52b122
Show file tree

Hide file tree

Showing 2 changed files with 26 additions and 27 deletions.
diff --git a/docs/platforms/javascript/common/best-practices/multiple-sentry-instances.mdx b/docs/platforms/javascript/common/best-practices/multiple-sentry-instances.mdx
@@ -37,18 +37,19 @@ import {
   Scope,
 } from "@sentry/browser";
 
-// filter integrations that use the global variable
-const integrations = getDefaultIntegrations({}).filter((defaultIntegration) => {
-  return !["BrowserApiErrors", "Breadcrumbs", "GlobalHandlers"].includes(
-    defaultIntegration.name
-  );
-});
 
 const client = new BrowserClient({
   dsn: "___PUBLIC_DSN___",
   transport: makeFetchTransport,
   stackParser: defaultStackParser,
-  integrations: integrations,
+  // filter integrations that use the global variable
+  integrations: (defaultIntegrations) =>
+    defaultIntegrations.filter(
+      (integration) =>
+        !["BrowserApiErrors", "Breadcrumbs", "GlobalHandlers"].includes(
+          integration.name
+        )
+    ),
 });
 
 const scope = new Scope();
@@ -89,20 +90,18 @@ function happyIntegration() {
   };
 }
 
-// filter integrations that use the global variable
-const integrations = Sentry.getDefaultIntegrations({}).filter(
-  (defaultIntegration) => {
-    return !["BrowserApiErrors", "Breadcrumbs", "GlobalHandlers"].includes(
-      defaultIntegration.name
-    );
-  }
-);
 
 const client1 = new Sentry.BrowserClient({
   dsn: "___PUBLIC_DSN___",
   transport: Sentry.makeFetchTransport,
   stackParser: Sentry.defaultStackParser,
-  integrations: [...integrations, happyIntegration()],
+  // filter integrations that use the global variable
+  integrations: (defaultIntegrations) => [
+    ...defaultIntegrations.filter(
+      (integration) => !["BrowserApiErrors", "Breadcrumbs", "GlobalHandlers"].includes(integration.name)
+    ),
+    happyIntegration(),
+  ],
   beforeSend(event) {
     console.log("client 1", event);
     return null; // Returning `null` prevents the event from being sent
@@ -115,7 +114,12 @@ const client2 = new Sentry.BrowserClient({
   dsn: "___PUBLIC_DSN___", // Can be a different DSN
   transport: Sentry.makeFetchTransport,
   stackParser: Sentry.defaultStackParser,
-  integrations: [...integrations, happyIntegration()],
+  integrations: (defaultIntegrations) => [
+    ...defaultIntegrations.filter(
+      (integration) => !["BrowserApiErrors", "Breadcrumbs", "GlobalHandlers"].includes(integration.name)
+    ),
+    happyIntegration(),
+  ],
   beforeSend(event) {
     console.log("client 2", event);
     return null; // Returning `null` prevents the event from being sent

diff --git a/docs/platforms/javascript/common/best-practices/shared-environments.mdx b/docs/platforms/javascript/common/best-practices/shared-environments.mdx
@@ -59,20 +59,15 @@ import {
   Scope,
 } from "@sentry/browser";
 
-// filter integrations that use the global variable
-const integrations = getDefaultIntegrations({}).filter((defaultIntegration) => {
-  return ![
-    "BrowserApiErrors",
-    "Breadcrumbs",
-    "GlobalHandlers",
-  ].includes(defaultIntegration.name);
-});
-
 const client = new BrowserClient({
   dsn: "___PUBLIC_DSN___",
   transport: makeFetchTransport,
   stackParser: defaultStackParser,
-  integrations: integrations,
+  // filter integrations that use the global variable
+  integrations: (defaultIntegrations) => 
+    defaultIntegrations.filter(
+      (integration) => !["BrowserApiErrors", "Breadcrumbs", "GlobalHandlers"].includes(integration.name)
+    ),
 });
 
 const scope = new Scope();