Skip to content

docs(platforms): Generalize query string collection wording#17756

Open
adinauer wants to merge 1 commit into
adinauer/docs/add-missing-query-string-clarificationsfrom
adinauer/docs/generalize-query-string-wording
Open

docs(platforms): Generalize query string collection wording#17756
adinauer wants to merge 1 commit into
adinauer/docs/add-missing-query-string-clarificationsfrom
adinauer/docs/generalize-query-string-wording

Conversation

@adinauer
Copy link
Copy Markdown
Member

Generalize the Apple and Native request query string descriptions so they do not imply query strings are only sent through tracing span attributes.

The data-collected pages should describe the data category rather than a single transport path. Query strings can also appear through other data paths such as breadcrumbs or event request data, so this switches those sections to the same path-agnostic wording used by the other SDK docs.

Avoid implying that Apple and Native query string collection only applies to tracing spans. Use more generic data-collected wording so breadcrumbs and event request data are not excluded by the description.

Co-Authored-By: Claude <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented May 13, 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 13, 2026 7:03pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
develop-docs Ignored Ignored May 13, 2026 7:03pm

Request Review

@codeowner-assignment codeowner-assignment Bot requested review from a team May 13, 2026 18:51
@adinauer adinauer mentioned this pull request May 13, 2026
5 tasks
@geoffg-sentry geoffg-sentry marked this pull request as ready for review May 13, 2026 19:24
Copy link
Copy Markdown
Member

@philprime philprime left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Member

@philprime philprime left a comment

Choose a reason for hiding this comment

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

LGTM

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.

2 participants