Skip to content

docs(js): remove Connect guide and integration (no longer supported) - #19513

Closed
mydea wants to merge 1 commit into
guide-cat/17-server-integration-matrixfrom
guide-cat/18-remove-connect
Closed

mydea wants to merge 1 commit into
guide-cat/17-server-integration-matrixfrom
guide-cat/18-remove-connect

Conversation

@mydea

@mydea mydea commented Sep 18, 2026

Copy link
Copy Markdown
Member

DESCRIBE YOUR PR

Stacked PR. Part of the JavaScript guide-categories series (#19492#19513). Depends on #19512; review/merge bottom-up.

The Sentry JavaScript SDK no longer supports the Connect framework, so remove its guide and the Connect integration page, and strip javascript.connect from every support list across the docs.

  • Delete guides/connect/ and configuration/integrations/connect.mdx.
  • Remove javascript.connect from 46 frontmatter/JSX support lists.
  • Repoint the two product getting-started samples from platform="javascript.connect" to javascript.node.
  • Redirect /platforms/javascript/guides/connect/* → the Node guide (Connect's former fallbackGuide).

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

@mydea
mydea added this pull request to stack #19507 September 18, 2026 11:29
@vercel

vercel Bot commented Sep 18, 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 Sep 22, 2026 1:25pm UTC
sentry-docs Error Error Sep 22, 2026 1:25pm UTC

Request Review

Comment thread redirects.js
Comment on lines 590 to +593
const userDocsRedirects = [
// Connect guide removed (SDK no longer supports Connect) → Node
{
source: '/platforms/javascript/guides/connect/:path*',

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.

Bug: Hard-coded links in the metrics and logs getting-started guides point to old connect paths. The new redirect rule sends users to non-existent pages, causing a 404 error.
Severity: HIGH

Suggested Fix

Update the hard-coded url props in /docs/product/metrics/getting-started/index.mdx and /docs/product/logs/getting-started/index.mdx. The links should point to the correct, existing pages within the /platforms/javascript/guides/node/ path, or be removed if no equivalent guide exists.

Prompt for AI Agent
Review the code at the location below. A potential bug has been identified by an AI
agent. Verify if this is a real issue. If it is, propose a fix; if not, explain why it's
not valid.

Location: redirects.js#L590-L593

Potential issue: The pull request introduces a redirect from
`/platforms/javascript/guides/connect/:path*` to
`/platforms/javascript/guides/node/:path*`. However, it fails to update hard-coded links
in two getting-started guides: `/docs/product/metrics/getting-started/index.mdx` and
`/docs/product/logs/getting-started/index.mdx`. These links still point to the old
`connect` paths. When a user clicks them, the redirect rule correctly fires but sends
them to non-existent pages (`/platforms/javascript/guides/node/metrics/` and
`/platforms/javascript/guides/node/logs/`), resulting in a 404 error and a broken user
flow.

Did we get this right? 👍 / 👎 to inform future reviews.

@mydea
mydea removed this pull request from stack #19507 September 22, 2026 13:07
@mydea
mydea added this pull request to stack #19556 September 22, 2026 13:08
The Sentry JavaScript SDK no longer supports the Connect framework, so remove
its guide and the Connect integration page, and strip `javascript.connect`
from every supported/notSupported list and platform sample across the docs.

- Delete `guides/connect/` and `configuration/integrations/connect.mdx`.
- Remove `javascript.connect` from 46 frontmatter/JSX support lists.
- Repoint the two product getting-started samples that used
  `platform="javascript.connect"` to `javascript.node`.
- Redirect `/platforms/javascript/guides/connect/*` → the Node guide
  (Connect's former fallbackGuide).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

This branch had an error being deployed

1 failed and 1 active deployments
Preview – develop-docs 94c4f622 Deployed Sep 22, 2026 by vercel[bot]
Preview – sentry-docs 94c4f622 Deployed Sep 22, 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.

2 participants