Skip to content

docs(js): Fix supported key on Ionic Capacitor source maps page - #19735

Open
mydea wants to merge 2 commits into
masterfrom
fn/js-ionic-capacitor-supported
Open

mydea wants to merge 2 commits into
masterfrom
fn/js-ionic-capacitor-supported

Conversation

@mydea

@mydea mydea commented Sep 30, 2026

Copy link
Copy Markdown
Member

DESCRIBE YOUR PR

The "Ionic Capacitor Build" source maps page (sourcemaps/uploading/ionic-capacitor.mdx) used Supported: with a capital S in its frontmatter. The docs build only reads supported, so the Capacitor-only restriction never applied, and the page showed up in the source maps sidebar of every JavaScript guide (React, Node, Next.js, …).

  • Fix the key to supported: so the page only renders for Capacitor.

The only link to this page comes from the Capacitor source maps overview (platform-includes/sourcemaps/overview/javascript.capacitor.mdx), so no links break. Other guides lose a page from their source maps sidebar that didn't apply to them.

IS YOUR CHANGE URGENT?

Help us prioritize incoming PRs by letting us know when the change needs to go live.
Select exactly one option. For deadlines, replace YYYY-MM-DD with the due date. You can update this information later by editing the PR description.

  • Urgent deadline (GA date, etc.): YYYY-MM-DD
  • Other deadline: YYYY-MM-DD
  • No deadline: 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 supplied a deadline.

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

🤖 Generated with Claude Code

The frontmatter used `Supported:` with a capital S, which the docs build
ignores. The Capacitor-only restriction never applied, so the Ionic
Capacitor Build page showed up for every JavaScript guide.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@vercel

vercel Bot commented Sep 30, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
develop-docs Ready Ready Preview Oct 1, 2026 1:58pm UTC
sentry-docs Ready Ready Preview Oct 1, 2026 1:58pm UTC

Request Review

@github-actions github-actions Bot added the Priority: Normal Docs review has no urgent deadline label Sep 30, 2026
@mydea
mydea marked this pull request as ready for review September 30, 2026 13:06
@mydea
mydea requested a review from a team September 30, 2026 13:06
@cursor

cursor Bot commented Sep 30, 2026

Copy link
Copy Markdown
Contributor

Rollouts skipped this pull request: Documentation-only change to an MDX docs page (frontmatter key fix).

Mention @change-monitor in a comment to monitor it anyway.

…guide

Limiting the Ionic Capacitor source maps page to the Capacitor guide
removes it from the JavaScript platform root and every other guide.
Redirect those URLs, and their .md exports, to the Capacitor guide's
page so existing links keep resolving.

Co-Authored-By: Shannon Anahata <shannon.anahata@sentry.io>

This branch was successfully deployed

2 active deployments
Preview – sentry-docs — 6e61b7ec Deployed Oct 1, 2026 by vercel[bot]
Preview – develop-docs — 6e61b7ec Deployed Oct 1, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Priority: Normal Docs review has no urgent deadline

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant