Repository navigation
feat(docs): Apply new quick start template to JavaScript quick start guides - #19983
Merged
Merged
Conversation
…id, and Gatsby Co-Authored-By: Shannon Anahata <shannon.anahata@sentry.io>
…Electron, Wasm, Astro, and React Router Co-Authored-By: Shannon Anahata <shannon.anahata@sentry.io>
…loudflare, Deno, Effect, and Elysia Co-Authored-By: Shannon Anahata <shannon.anahata@sentry.io>
…, Hono, NestJS, and Nitro Co-Authored-By: Shannon Anahata <shannon.anahata@sentry.io>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
1 Skipped Deployment
|
Contributor
|
Rollout plan could not be created because the planning run failed. Mention |
1 of 6 tasks
…k start guide Co-Authored-By: Shannon Anahata <shannon.anahata@sentry.io>
Co-Authored-By: Shannon Anahata <shannon.anahata@sentry.io>
1 of 6 tasks
sentry-junior Bot
added a commit
that referenced
this pull request
Oct 15, 2026
The quick start template PRs (#19980-#19983) still have to remove the SplitLayout wrappers from 38 files. Until they merge, render those tags as plain pass-throughs so the build and preview succeed. Remove this shim once no MDX file uses the wrappers. Co-Authored-By: Shannon Anahata <shannon.anahata@sentry.io>
sfanahata
added a commit
that referenced
this pull request
Oct 19, 2026
…20017) ## DESCRIBE YOUR PR Removes the `SplitLayout` component everywhere. Pages that used it now render their text and code as regular docs content, with the standard docs spacing. - Removes the `<SplitLayout>`, `<SplitSection>`, `<SplitSectionText>`, and `<SplitSectionCode>` wrapper lines from all 169 MDX files that used them, including the quick start guides. Only the tag lines and the blank lines around them change. Parsing each file before and after shows the same content once the wrappers are ignored. - Deletes `src/components/splitLayout/` and its registration in `src/mdxComponents.ts`. - Drops the SplitLayout guidance from `.claude/skills/technical-docs/SKILL.md`. Step numbering is unaffected. `StepConnector` finds its `h2` headings with a descendant selector, so it doesn't depend on these wrappers. This replaces #19984, which made SplitLayout render as a single column. ### Merge order Merge this **first**. After it merges, the quick start template PRs that also edit these files (#19980 Ruby, #19981 Dart and Flutter, #19982 Python, #19983 JavaScript) will need `master` merged in. Those conflicts are limited to the removed wrapper tags. ## IS YOUR CHANGE URGENT? - [ ] Urgent deadline (GA date, etc.): YYYY-MM-DD - [ ] Other deadline: YYYY-MM-DD - [x] No deadline: Not urgent, can wait up to 1 week+ ## PRE-MERGE CHECKLIST - [ ] 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) <!-- junior-request-attribution:start --> via **shannon.anahata**. <!-- junior-request-attribution:end --> <!-- junior-session-footer:start --> <!-- junior-conversation-id:slack%3AD0BB8KQT6MU%3A1792013336.218129 --> -- [View Junior Session](https://junior-prod.sentry.dev/conversations/slack%3AD0BB8KQT6MU%3A1792013336.218129) [[Sentry]](https://sentry.sentry.io/explore/conversations/slack%3AD0BB8KQT6MU%3A1792013336.218129/?project=4510944073809921) <!-- junior-session-footer:end --> --------- Co-authored-by: sentry-junior[bot] <264270552+sentry-junior[bot]@users.noreply.github.com> Co-authored-by: Shannon Anahata <shannon.anahata@sentry.io>
Resolve conflicts from #20017 (SplitLayout removal): keep this branch's content without the removed wrapper tags, and keep master's other changes to the same files. Co-Authored-By: Shannon Anahata <shannon.anahata@sentry.io>
…uides The Bun and Deno runtime metrics integration pages only exist under the Bun and Deno guides, so the platform-relative links 404 on Hono.
inventarSarah
force-pushed
the
docs/quickstart-template-javascript
branch
from
October 21, 2026 11:40
8470b36 to
d922833
Compare
added 2 commits
October 21, 2026 10:48
Remove the onboarding option checkboxes and include every supported feature in the init snippets. Align the page with the wizard's current output, add Send Logs and Send Application Metrics steps, and add javascript.nextjs to the quick start feature expandables.
sfanahata
approved these changes
Oct 21, 2026
sfanahata
left a comment
Contributor
There was a problem hiding this comment.
All major frameworks built on the production Ember example. Reviewed thoroughly by Sarah and myself. Ready to merge.
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
DESCRIBE YOUR PR
Applies the quick start template from #19295 (Ember) to the rest of the JavaScript quick starts:
/platforms/javascript/), React, Vue, Svelte, Solid, SolidStart, Gatsby, Capacitor, Electron, WasmChanges on each page:
___PRODUCT_OPTION___markers.Features are listed per SDK. Profiling appears only where the guide offered it, and Session Replay and User Feedback appear only on browser SDKs.
Plain JavaScript page:
/platforms/javascript/is rendered fromplatform-includes/getting-started-complete/javascript.mdx, which only that page uses. Its init snippet (npm, Loader and CDN tabs) and Verify snippet are written inline in that file, so the sharedgetting-started-configandgetting-started-verifysnippets other pages use are unchanged. The CDN tab now loads the full bundle (bundle.tracing.replay.feedback.logs.metrics.min.js), because the old tracing-only bundle doesn't include Session Replay or User Feedback. javascript.nextjs has been added to includes/quick-start-features-expandable.mdx and includes/quick-start-locate-data-expandable.mdx.IS YOUR CHANGE URGENT?
PRE-MERGE CHECKLIST
via shannon.anahata.
--
View Junior Session [Sentry]