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.

Sign up for GitHub

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

Jump to bottom

Fix canonical addresses in the documentation #8886

Closed
krzysztofspilka opened this issue Oct 20, 2021 · 1 comment · Fixed by #8905
Closed

Fix canonical addresses in the documentation #8886

krzysztofspilka opened this issue Oct 20, 2021 · 1 comment · Fixed by #8905
Assignees
@jansiegel
Labels
Docs: Content Issues related to Handsontable's documentation content enhancement Status: Released

Comments

@krzysztofspilka
Copy link
Member

krzysztofspilka commented Oct 20, 2021 •
edited

Description

We migrated our documentation to a new engine, VuePress, but forgot about canonical tags. They should point at the latest official revision so Google knows that all other revisions (versions) are just copies, and it's not worthy crawling them.

Versions 9.x and later (VuePress)

Guides

<link rel="canonical" href="https://handsontable.com/docs/{slug}" />

The Guide Introduction page doesn't have a slug so it should always get the address:

<link rel="canonical" href="https://handsontable.com/docs/" />

API

<link rel="canonical" href="https://handsontable.com/docs/api/{slug}" />

The API Introduction page doesn't have a slug so it should always get the address:

<link rel="canonical" href="https://handsontable.com/docs/api" />

Examples

<link rel="canonical" href="https://handsontable.com/docs/examples/{slug}" />

The Examples Introduction page doesn't have a slug so it should always get the address:

<link rel="canonical" href="https://handsontable.com/docs/examples" />

For instance, at:
https://handsontable.com/docs/9.0/api/bind-rows-with-headers/
there will be this tag:
<link rel="canonical" href="https://handsontable.com/docs/api/bind-rows-with-headers" />

Versions from 4.0.0 to 8.4.0

Recommend robots to not crawl these old pages by adding:

<meta name="robots" content="noindex, nofollow">

If in doubt where to put it in the code, head to specs at https://developers.google.com/search/docs/advanced/robots/robots_meta_tag
@krzysztofspilka krzysztofspilka added Docs: Content Issues related to Handsontable's documentation content enhancement labels Oct 20, 2021
jansiegel added a commit that referenced this issue Oct 26, 2021
@jansiegel
- Remove a problematic fragment of code causing a duplicated url in t…
2165106 
…he rel=canonical path

- Fix a double-slash issue in the rel=canonical paths #8886
@jansiegel jansiegel mentioned this issue Oct 26, 2021
Merged
7 tasks
@krzysztofspilka krzysztofspilka linked a pull request Nov 3, 2021 that will close this issue
Fix issues with new documentation's rel=canonical elements. #8905
Merged
7 tasks
jansiegel added a commit that referenced this issue Nov 4, 2021
@jansiegel
Fix issues with new documentation's rel=canonical elements. (#8905)
0b3000b 
* - Remove a problematic fragment of code causing a duplicated url in the rel=canonical path
- Fix a double-slash issue in the rel=canonical paths #8886

* Add the changelog entry.
@AMBudnik AMBudnik mentioned this issue Nov 18, 2021
Closed
@AMBudnik
Copy link
Contributor

Issue closed as fixed in Handsontable v11.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jansiegel jansiegel

Labels
Docs: Content Issues related to Handsontable's documentation content enhancement Status: Released
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix issues with new documentation's rel=canonical elements. handsontable/handsontable
4 participants
@jansiegel @krzysztofspilka @AMBudnik @kirszenbaum