-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ref(javascript): Improve beforeSendSpan
documentation
#10907
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Skipped Deployment
|
@@ -213,13 +213,15 @@ If the transaction currently being processed has a parent transaction (from an u | |||
|
|||
Learn more about <PlatformLink to="/configuration/sampling/">configuring the sample rate</PlatformLink>. | |||
|
|||
### Using <PlatformIdentifier name="before-send-transaction" /> | |||
### Using `beforeSendTransaction` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I changed this to the plain string because 1. I don't think <PlatformIdentifier>
does anything here since it's a JS-specific docs page and 2. I can't link to this heading properly because of #10613.
Happy to wait with merging this PR until the issue is fixed but I still don't see why we need PlatformIdentifier at all here.
Bundle ReportChanges will increase total bundle size by 206.85kB ⬆️
|
Gonna merge this because it came up today again in an issue. |
This PR changes the `beforeSendSpan` JSDoc analogously to the docs change (getsentry/sentry-docs#10907). It now more clearly points out that `beforeSendSpan` is only called for child spans but not for root spans. ref #13119
This small PR adds a sentence to the existing
beforeSendSpan
documentation that the callback is only applied to child spans andbeforeSendTransaction
should be used to drop entire span trees/transactions. Also reworded the description slightly.Making this change because users seemed to be confused about when
beforeSendSpan
is applied.