Skip to content

Fix shared docs links - #2131

Draft
taras-yemets wants to merge 2 commits into
mainfrom
fix-shared-docs-links
Draft

taras-yemets wants to merge 2 commits into
mainfrom
fix-shared-docs-links

Conversation

@taras-yemets

@taras-yemets taras-yemets commented Nov 3, 2026 •

Copy link
Copy Markdown
Contributor

Adds the headings and redirect that product and generated links from sourcegraph/sourcegraph now point to, so they work on this site until cutover. See #2128 (review).

The link check failure is expected: the redirect changes how it reports an existing absolute link in the generated alerts.mdx, which the next sync would overwrite if edited here.

@vercel

vercel Bot commented Nov 3, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
sourcegraph-docs Ready Ready Preview Nov 3, 2026 6:38am UTC

Request Review

Add the headings and redirect that links generated in
sourcegraph/sourcegraph now point to, so they resolve on this site until
cutover.
@taras-yemets
taras-yemets force-pushed the fix-shared-docs-links branch from 6ea4ba1 to eb6f4b1 Compare November 3, 2026 06:15
@github-actions

github-actions Bot commented Nov 3, 2026 •

Copy link
Copy Markdown
Contributor

Direct preview links to pages changed in this PR:

@github-actions

github-actions Bot commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

@github-actions

github-actions Bot commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

❌ This PR introduces 1 broken link(s)

Absolute links

Write links on this site as relative paths (/admin/config/site-config), not https://sourcegraph.com/docs/…: absolute links leave the preview deployment and local dev server, and hide moved pages behind redirects.

docs/self-hosted/observability/alerts.mdx

  • line 5568
    • Link: https://sourcegraph.com/docs/self-hosted/deploy/search-performance#shard-merging
    • Problem: Absolute link to a moved page
    • Fix: /admin/search#shard-merging

Reproduce locally with pnpm check links --check-anchors --check-self-links (see dev/check-links.mjs).

Adding a redirect in src/data/redirects.ts does not satisfy this check, because it’s a workaround instead of a fix.

@github-actions

github-actions Bot commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

⚠️ Spell check found 1 issue(s) in this PR

Only findings on lines added by this PR, and in its title and description, are shown.

Pull request description

  • line 1, column 140
    • cutover → outover

Please correct the spelling, or add words which are correct to cspell-allow-list.txt.

Comment on this PR with /spell <list of words>, and a bot will push a commit to your branch adding your comment's list of words to cspell-allow-list.txt for you.

To accept all of the words above as correct: /spell cutover

This branch was successfully deployed

1 active deployment
Preview — 7aaa8f4b Deployed Nov 3, 2026 by vercel[bot]
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.

1 participant