Skip to content

docs: Update links to use sourcegraph.com/docs#1122

Closed
trly wants to merge 1 commit into
mainfrom
docs/fix-docs-url
Closed

docs: Update links to use sourcegraph.com/docs#1122
trly wants to merge 1 commit into
mainfrom
docs/fix-docs-url

Conversation

@trly

@trly trly commented May 7, 2025

Copy link
Copy Markdown
Contributor

update links in the documentation to point to the sourcegraph.com/docs domain instead of docs.sourcegraph.com.

Pull Request approval

You will need to get your PR approved by at least one member of the Sourcegraph team. For reviews of docs formatting, styles, and component usage, please tag the docs team via the #docs Slack channel.

update links in the documentation to point to the `sourcegraph.com/docs` domain instead of `docs.sourcegraph.com`.
@trly trly requested review from MaedahBatool and burmudar May 7, 2025 14:43
@vercel

vercel Bot commented May 7, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
sourcegraph-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 7, 2025 2:46pm

@burmudar burmudar left a comment

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.

The old links do work - the reason they didn't was due to a Cloudflare rule regressions.
The new links give 404s

For context

@trly

trly commented May 7, 2025

Copy link
Copy Markdown
Contributor Author

This change was related to this thread: https://sourcegraph.slack.com/archives/C01DXLN3D0T/p1746555762860679?thread_ts=1746554952.599839&cid=C01DXLN3D0T

If the URL updates aren't actually needed this can be discarded. Up to the docs team :)

@burmudar

burmudar commented May 7, 2025

Copy link
Copy Markdown
Contributor

The old links do work - the reason they didn't was due to a Cloudflare rule regressions. The new links give 404s

For context

I am in the process of fixing the infra that caused this

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.

2 participants