Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[DOCS] documentation site 'Edit On GitHub' links return a 404 #2863

Closed
wesdevpro opened this issue Jan 25, 2024 Discussed in #2859 · 7 comments
Closed

[DOCS] documentation site 'Edit On GitHub' links return a 404 #2863

wesdevpro opened this issue Jan 25, 2024 Discussed in #2859 · 7 comments
Labels
Status: Stale Used by stalebot to clean house

Comments

@wesdevpro
Copy link
Contributor

Discussed in #2859

Originally posted by benhopkinstech January 22, 2024
Hello all,

Sorry if this is the wrong means of doing this but I don't feel it warranted an issue being created due to it not being a problem with the code itself.

When I was browsing the documentation website I noticed that the 'Edit on GitHub' links all seem to return a 404.

The links seem to point to the repository on the master branch which must have existed previously before being renamed to main.

For example https://github.com/octokit/octokit.net/edit/master/docs/index.md is the link provided when trying to edit the home page.

This should read https://github.com/octokit/octokit.net/edit/main/docs/index.md.

Hopefully it should just be a matter of updating the base url on the documentation site to https://github.com/octokit/octokit.net/edit/main/docs/.

Ben

@kfcampbell kfcampbell moved this from 🆕 Triage to 🔥 Backlog in 🧰 Octokit Active Feb 2, 2024
@kfcampbell
Copy link
Member

@benhopkinstech would you prefer to have the conversation here or in the discussion you created?

@benhopkinstech
Copy link

@kfcampbell just keep it here and close the discussion, I wasn't sure where to raise it initially as I know the site is external to the repository.

@kfcampbell
Copy link
Member

I've done so! I'm also going to copy my initial response there to here:

What's really curious to me is that in most cases, such as clicking on the "Getting Started" guide here, GitHub will redirect to the correct branch and show a little banner:

image

As for actually fixing this, I'm struggling to find where to make the change. I'm not seeing any relevant configuration in our docs configuration in mkdocs.yml. Searching for "master" in our codebase doesn't lead anywhere that looks useful (though incidentally I created this PR to correct some links using it when they shouldn't be). I wonder if readthedocs.io assumes our default branch incorrectly?

@benhopkinstech
Copy link

I've requested approval on that PR.

It still doesn't answer the issue on the readthedocs site, I have found this issue.

Looks like we will need whoever has an account for the site to go in and change the settings.

@kfcampbell
Copy link
Member

Ooh that's interesting... @nickfloyd do you know who might own those credentials? I've checked and I don't have access to anything that looks relevant.

@benhopkinstech
Copy link

Sign in should be available here

Copy link

👋 Hey Friends, this issue has been automatically marked as stale because it has no recent activity. It will be closed if no further activity occurs. Please add the Status: Pinned label if you feel that this issue needs to remain open/active. Thank you for your contributions and help in keeping things tidy!

@github-actions github-actions bot added the Status: Stale Used by stalebot to clean house label Oct 30, 2024
@github-actions github-actions bot closed this as completed Nov 7, 2024
@github-project-automation github-project-automation bot moved this from 🔥 Backlog to ✅ Done in 🧰 Octokit Active Nov 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Stale Used by stalebot to clean house
Projects
Archived in project
Development

No branches or pull requests

3 participants