Skip to content

js(docs): Update Tracing pages for stream mode#17862

Open
inventarSarah wants to merge 9 commits into
masterfrom
smi/span-first/js-tracing-updates
Open

js(docs): Update Tracing pages for stream mode#17862
inventarSarah wants to merge 9 commits into
masterfrom
smi/span-first/js-tracing-updates

Conversation

@inventarSarah
Copy link
Copy Markdown
Collaborator

@inventarSarah inventarSarah commented May 22, 2026

DESCRIBE YOUR PR

This branch contains light edits to the JavaScript Tracing pages, mentioning the New Spans and Stream Mode as needed.

  • added in-line mentions of stream mode on pages that only sparingly feature transaction-focused content
  • added an Alert at the top of pages re. stream mode on pages that talk a lot about transactions & their concepts

Important

Must only be merged after #17834
This branch is separate from this PR and thus does not contain the new /tracing/new-spans page. However, this branch adds links to that page, which the 404 Linter identifies correctly as broken

Closes: #17836

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.):
  • Other deadline: End of May
  • 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

EXTRA RESOURCES

@vercel
Copy link
Copy Markdown

vercel Bot commented May 22, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
sentry-docs Ready Ready Preview, Comment May 27, 2026 8:56am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
develop-docs Ignored Ignored Preview May 27, 2026 8:56am

Request Review

});
```

```javascript {tabTitle:Stream Mode}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm liking these additions!

Comment thread docs/platforms/javascript/common/tracing/troubleshooting/index.mdx Outdated

See <PlatformLink to="/tracing/new-spans/">New Spans</PlatformLink> for more information.

## `ignoreSpans` Rules No Longer Work After Migrating to Stream Mode
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do they no longer work, or does the behavior change?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated this to "... No Longer Work As Expected After Migrating ..."

Copy link
Copy Markdown
Contributor

@sfanahata sfanahata left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good from a content perspective. I mostly wonder about revealing the 1k limit by explicit number vs just saying stream mode is more efficient in capturing the same data? Thoughts @cleptric?

inventarSarah and others added 4 commits May 27, 2026 10:39
Co-authored-by: Shannon Anahata <shannon.anahata@gmail.com>
…try/sentry-docs into smi/span-first/js-tracing-updates
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.

Update JS SDK Tracing docs for stream mode/new spans

2 participants