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

Use correct canonical URL while building docs #6829

Merged
merged 1 commit into from
Apr 5, 2023
Merged

Conversation

ashwinvis
Copy link
Contributor

As noted in readthedocs/readthedocs.org#10224 with the current configuration the canonical URL is wrong with different sphinx builds. For instance the "latest" docs point to "stable". This leads to apps like Mastodon to generate previews to pages which don't exist since they rely on

<link rel="canonical" href="https://jupyter-notebook.readthedocs.io/en/stable/notebook_7_features.html" />

This should fix that.

As noted in readthedocs/readthedocs.org#10224 with the current configuration the canonical URL is wrong with different sphinx builds. For instance the "latest" docs point to "stable". This leads to apps like Mastodon to generate previews to pages which don't exist since they rely on

    <link rel="canonical" href="https://jupyter-notebook.readthedocs.io/en/stable/notebook_7_features.html" />

This should fix that.
@github-actions
Copy link
Contributor

github-actions bot commented Apr 5, 2023

Binder 👈 Launch a Binder on branch ashwinvis/notebook/patch-1

@benjaoming
Copy link

The PR build looks good 👍

https://jupyter-notebook--6829.org.readthedocs.build/en/6829/

<link rel="canonical" href="https://jupyter-notebook--6829.org.readthedocs.build/en/6829/index.html" />

@jtpio jtpio added this to the 7.0 milestone Apr 5, 2023
Copy link
Member

@jtpio jtpio left a comment

Choose a reason for hiding this comment

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

Thanks!

@jtpio jtpio merged commit 6e60c72 into jupyter:main Apr 5, 2023
jtpio added a commit that referenced this pull request Apr 5, 2023
* Use correct canonical URL while building docs (#6829)

As noted in readthedocs/readthedocs.org#10224 with the current configuration the canonical URL is wrong with different sphinx builds. For instance the "latest" docs point to "stable". This leads to apps like Mastodon to generate previews to pages which don't exist since they rely on

    <link rel="canonical" href="https://jupyter-notebook.readthedocs.io/en/stable/notebook_7_features.html" />

This should fix that.

* Renaming migration file to `migrating_to_notebook7`

* Use the correct name

* fix ref

---------

Co-authored-by: Ashwin V. Mohanan <9155111+ashwinvis@users.noreply.github.com>
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 5, 2024
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.

None yet

3 participants