octokit.net icon indicating copy to clipboard operation
octokit.net copied to clipboard

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

Open wesdevpro opened this issue 1 year ago • 6 comments

Discussed in https://github.com/octokit/octokit.net/discussions/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

wesdevpro avatar Jan 25 '24 20:01 wesdevpro

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

kfcampbell avatar Feb 02 '24 19:02 kfcampbell

@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.

benhopkinstech avatar Feb 02 '24 19:02 benhopkinstech

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?

kfcampbell avatar Feb 02 '24 19:02 kfcampbell

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.

benhopkinstech avatar Feb 02 '24 20:02 benhopkinstech

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.

kfcampbell avatar Feb 02 '24 21:02 kfcampbell

Sign in should be available here

benhopkinstech avatar Feb 02 '24 21:02 benhopkinstech