Skip to content

Commit 66c08c2

Browse files
authored
docs(javascript): Add list of integrations that use globals (#16384)
## DESCRIBE YOUR PR Documents missing integrations to filter. Ref: getsentry/sentry-javascript#19324 ## IS YOUR CHANGE URGENT? Help us prioritize incoming PRs by letting us know when the change needs to go live. - [ ] Urgent deadline (GA date, etc.): <!-- ENTER DATE HERE --> - [ ] Other deadline: <!-- ENTER DATE HERE --> - [ ] None: Not urgent, can wait up to 1 week+ ## SLA - Teamwork makes the dream work, so please add a reviewer to your PRs. - Please give the docs team up to 1 week to review your PR unless you've added an urgent due date to it. Thanks in advance for your help! ## PRE-MERGE CHECKLIST *Make sure you've checked the following before merging your changes:* - [ ] Checked Vercel preview for correctness, including links - [ ] PR was reviewed and approved by any necessary SMEs (subject matter experts) - [ ] PR was reviewed and approved by a member of the [Sentry docs team](https://github.com/orgs/getsentry/teams/docs)
1 parent 3283775 commit 66c08c2

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

docs/platforms/javascript/common/best-practices/shared-environments.mdx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ When setting up Sentry in a shared environment where multiple Sentry instances m
4949
## Shared Environment Setup
5050

5151
For the use cases listed above, set up a client manually as seen in the example below.
52-
In addition, avoid adding any integrations that use global state, like `Breadcrumbs` or `GlobalHandlers`.
52+
In addition, avoid adding any integrations that use global state, like `Breadcrumbs` or `GlobalHandlers` (see code snippet below).
5353
Furthermore, some default integrations that use the global state have to be filtered as in the example below.
5454
In these scenarios, it's a best practice to avoid using any integrations and to rely on manually capturing errors.
5555

@@ -64,7 +64,7 @@ import {
6464

6565
// filter integrations that use the global variable
6666
const integrations = getDefaultIntegrations({}).filter((defaultIntegration) => {
67-
return !["BrowserApiErrors", "Breadcrumbs", "GlobalHandlers"].includes(
67+
return !["BrowserApiErrors", "BrowserSession", "Breadcrumbs", "ConversationId", "GlobalHandlers", "FunctionToString"].includes(
6868
defaultIntegration.name
6969
);
7070
});

0 commit comments

Comments
 (0)