Skip to content

Comments

docs(profiling): Fix missing platform item header for profile_chunk#16135

Merged
s1gr1d merged 2 commits intomasterfrom
sig/chunk-platform
Jan 23, 2026
Merged

docs(profiling): Fix missing platform item header for profile_chunk#16135
s1gr1d merged 2 commits intomasterfrom
sig/chunk-platform

Conversation

@s1gr1d
Copy link
Member

@s1gr1d s1gr1d commented Jan 23, 2026

DESCRIBE YOUR PR

Adds missing platform item header.

closes https://linear.app/getsentry/issue/JS-1539/document-platform-for-profile-chunk (internal Linear)

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:
  • 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

@linear
Copy link

linear bot commented Jan 23, 2026

@vercel
Copy link

vercel bot commented Jan 23, 2026

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

Project Deployment Review Updated (UTC)
develop-docs Ready Ready Preview, Comment Jan 23, 2026 0:05am
1 Skipped Deployment
Project Deployment Review Updated (UTC)
sentry-docs Ignored Ignored Preview Jan 23, 2026 0:05am

Request Review

_None_
`platform`

: **String, required.** The platform of the profiled application. This item header is used for rate limiting and categorization. Examples: `"javascript"`, `"node"`, `"python"`, `"cocoa"`, etc.
Copy link
Member

Choose a reason for hiding this comment

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

Could also mention that it needs to match the platform in the payload

@s1gr1d s1gr1d enabled auto-merge (squash) January 23, 2026 12:01
@s1gr1d s1gr1d merged commit e6477b6 into master Jan 23, 2026
14 checks passed
@s1gr1d s1gr1d deleted the sig/chunk-platform branch January 23, 2026 12:05
@github-actions github-actions bot locked and limited conversation to collaborators Feb 8, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants