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

Fix link to extending Jupyterhub page #723

Merged
merged 2 commits into from
Jun 14, 2018
Merged

Conversation

alanderex
Copy link
Contributor

No description provided.

@minrk minrk mentioned this pull request Jun 14, 2018
@@ -164,6 +164,6 @@ Install JupyterHub
entering any username and password combination will let you enter the hub.

Congratulations! Now that you have JupyterHub running, you can
`extend it <extending-jupyterhub.html>`_ in many ways. You can use a pre-built
`extend it <extending-jupyterhub.rst>`_ in many ways. You can use a pre-built
Copy link
Member

Choose a reason for hiding this comment

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

The target in the built docs is html, not rst, so this change would break the link.

However, these links should be using rst refs, not file extensions, so this should be:

:ref:`extend it <extending-jupyterhub>`

There appear to be several that should be fixed in this doc. Would you like to take a stab? You can run make linkcheck to verify that links resolve.

@choldgraf choldgraf merged commit bfe71e5 into jupyterhub:master Jun 14, 2018
@choldgraf
Copy link
Member

thanks for the fix @alanderex !

@manics manics mentioned this pull request Aug 15, 2018
7 tasks
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.

3 participants