Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

Conversation

@BolajiOlajide
Copy link
Contributor

Test plan

All links to the old docs page docs.sourcegraph.com should be updated to their sourcegraph.com/docs equivalent.

Copy link
Member

@camdencheek camdencheek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oof. Looks like there are more docs.sourcegraph.com links around the codebase as well. Should these be pointing to the docs for the current sourcegraph version? Right now, this just directs the user to the latest version. The /help/... route will redirect to the docs corresponding to the instance version.

@BolajiOlajide
Copy link
Contributor Author

Oof. Looks like there are more docs.sourcegraph.com links around the codebase as well. Should these be pointing to the docs for the current sourcegraph version? Right now, this just directs the user to the latest version. The /help/... route will redirect to the docs corresponding to the instance version.

Ideally yes, but a lot of this change aren't easy to migrate without a manual check. Especially because the new docs site url structure isn't the same as the old.

I prioritized the new links added by the licensing work because it felt like the smallest change I could make this close to the release and with fewer stakeholders involved.

@BolajiOlajide BolajiOlajide merged commit 99656c1 into main Feb 9, 2024
@BolajiOlajide BolajiOlajide deleted the bo/tr/update-links-new-docs branch February 9, 2024 18:17
sourcegraph-release-bot pushed a commit that referenced this pull request Feb 9, 2024
keegancsmith pushed a commit that referenced this pull request Feb 11, 2024
…0378)

licensing: update doc links to the new doc website (#60376)

update links

(cherry picked from commit 99656c1)

Co-authored-by: Bolaji Olajide <25608335+BolajiOlajide@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants