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

Adding correct Github project links #2210

Closed
wants to merge 1 commit into from

Conversation

KhushbuB
Copy link

@waylan
Copy link
Member

waylan commented Oct 27, 2020

That link is specifically for GitHub's enterprise server. Shouldn't we be linking to the docs for free pro team?

Note, I have not yet reviewed the content of the linked page and therefore have not confirmed that it is the correct page regardless of which GitHub serve is linked.

@ktomk
Copy link
Contributor

ktomk commented Nov 5, 2020

Reported this upstream and it is already fixed with a redirect in place:

$ curl -Ls -o /dev/null -w '%{url_effective}\n' https://help.github.com/articles/creating-project-pages-manually/ 
https://docs.github.com/en/free-pro-team@latest/github/working-with-github-pages/creating-a-github-pages-site

Fix: github/docs@5536f92
PR: github/docs#1097
Ref: https://github.com/github/docs/discussions/1071

@@ -357,7 +357,7 @@ the MkDocs IRC channel `#mkdocs` on freenode.
[nav]: user-guide/configuration/#nav
[discussion group]: https://groups.google.com/forum/#!forum/mkdocs
[GitHub issues]: https://github.com/mkdocs/mkdocs/issues
[GitHub project pages]: https://help.github.com/articles/creating-project-pages-manually/
[GitHub project pages]: https://docs.github.com/en/enterprise-server@2.22/github/working-with-github-pages/creating-a-github-pages-site
Copy link
Member

Choose a reason for hiding this comment

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

Shouldn't this be changed to https://docs.github.com/en/free-pro-team@latest/github/working-with-github-pages/creating-a-github-pages-site?

Copy link
Contributor

Choose a reason for hiding this comment

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

From the conversation and the change-set upstream, my understanding is that the URI should be changed to:

  • https://docs.github.com/github/working-with-github-pages/creating-a-github-pages-site

@waylan
Copy link
Member

waylan commented Nov 6, 2020

I took a look at the content of the new page and it is not at all the same content as we linked to before. According to the wayback machine, most recently the link forwarded to Creating Project Pages Using the Command Line, which is nearly identical to the old page from when the link was first created: Creating Project Pages manually.

The content of the old page is not too long. I suspect it would make more sense to simply replicate the instructions from the old page in our own documentation. Of course, we should adapt it to be MkDocs specific and be careful not to just copy the old text.

@ktomk
Copy link
Contributor

ktomk commented Nov 6, 2020

Yes, good thinking to use wayback machine for this, didn't check that.

And I'm seeing that on the index it is giving pointers for hosting products (GitHub Pages and Amazon S3), I'd suggest to turn "GitHub project pages" into "GitHub Pages" (wording) as it looks like the correct name to me. It is also in use in Mkdocs docs already: https://www.mkdocs.org/user-guide/deploying-your-docs/#github-pages .

Instead of giving command line instructions about git (e.g. how to create a branch) an alternative that looks good to me is to link
to the product page for GitHub Pages, of which I could gather two:

Can't say which of those two would be the better choice for Mkdocs visitors, maybe the one from the docs?

And when doing so, promoting Mkdocs built-in gh-pages command with a link to the "Deploying your Docs" page. So it's not all only to the outside when there is Mkdocs content about GitHub Pages support.

@waylan
Copy link
Member

waylan commented Nov 6, 2020

I just realized this link is at the end of the "Getting Started" guide. I thought this was on the deployment docs page. That being the case, yes, a more generic link pointing to the product page (https://pages.github.com/) makes more sense. For details users can follow the link from there to Deploying your Docs.

waylan added a commit to waylan/mkdocs that referenced this pull request Nov 6, 2020
We already link to the Deploying for Docs page which includes specific
instructions for various hosts. There is no need to also link to those
hosts in the Getting Started guide.

Closes mkdocs#2210, closes mkdocs#2104, closes mkdocs#1494.
@waylan waylan closed this in #2224 Nov 6, 2020
waylan added a commit that referenced this pull request Nov 6, 2020
We already link to the Deploying for Docs page which includes specific
instructions for various hosts. There is no need to also link to those
hosts in the Getting Started guide.

Closes #2210, closes #2104, closes #1890.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants