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

Add canonical URL to all pages #4473

Merged
merged 2 commits into from
Jul 18, 2023
Merged

Conversation

jdbaldry
Copy link
Member

Grafana Cloud documentation wants to reuse this documentation as part of its reorganization.

Having duplicate pages can be detrimental to SEO and setting the canonical URL helps by letting search engines only index the canonical page.

We do not index /docs/agent/next/ pages, only /docs/agent/latest/, so there is no need to concern ourselves with how this canonical is managed across versions. Pages that are not "latest", automatically have their canonical set to "latest" by the website.

I will open a similar PR in the latest version branch to update the current "latest" pages.

Grafana Cloud documentation wants to reuse this documentation as part of its reorganization.

Having duplicate pages can be detrimental to SEO and setting the canonical URL helps by letting search engines only index the canonical page.

We do not index /docs/agent/next/ pages, only /docs/agent/latest/, so there is no need to concern ourselves with how this canonical is managed across versions.
Pages that are not "latest", automatically have their canonical set to "latest" by the website.

I will open a similar PR in the latest version branch to update the current "latest" pages.

Signed-off-by: Jack Baldry <jack.baldry@grafana.com>
Copy link
Contributor

@clayton-cornell clayton-cornell left a comment

Choose a reason for hiding this comment

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

LGTM

Signed-off-by: Jack Baldry <jack.baldry@grafana.com>
@jdbaldry jdbaldry requested a review from a team as a code owner July 18, 2023 08:48
Copy link
Collaborator

@mattdurham mattdurham left a comment

Choose a reason for hiding this comment

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

LGTM

@mattdurham mattdurham merged commit 6f3fe5f into main Jul 18, 2023
6 checks passed
@mattdurham mattdurham deleted the jdb/2023-07-add-canonical-URLs branch July 18, 2023 13:01
clayton-cornell pushed a commit that referenced this pull request Aug 14, 2023
* Add canonical URL to all pages

Grafana Cloud documentation wants to reuse this documentation as part of its reorganization.

Having duplicate pages can be detrimental to SEO and setting the canonical URL helps by letting search engines only index the canonical page.

We do not index /docs/agent/next/ pages, only /docs/agent/latest/, so there is no need to concern ourselves with how this canonical is managed across versions.
Pages that are not "latest", automatically have their canonical set to "latest" by the website.

I will open a similar PR in the latest version branch to update the current "latest" pages.

Signed-off-by: Jack Baldry <jack.baldry@grafana.com>

* Reverse changes into template

Signed-off-by: Jack Baldry <jack.baldry@grafana.com>

---------

Signed-off-by: Jack Baldry <jack.baldry@grafana.com>
clayton-cornell pushed a commit that referenced this pull request Aug 14, 2023
* Add canonical URL to all pages

Grafana Cloud documentation wants to reuse this documentation as part of its reorganization.

Having duplicate pages can be detrimental to SEO and setting the canonical URL helps by letting search engines only index the canonical page.

We do not index /docs/agent/next/ pages, only /docs/agent/latest/, so there is no need to concern ourselves with how this canonical is managed across versions.
Pages that are not "latest", automatically have their canonical set to "latest" by the website.

I will open a similar PR in the latest version branch to update the current "latest" pages.

Signed-off-by: Jack Baldry <jack.baldry@grafana.com>

* Reverse changes into template

Signed-off-by: Jack Baldry <jack.baldry@grafana.com>

---------

Signed-off-by: Jack Baldry <jack.baldry@grafana.com>
@github-actions github-actions bot added the frozen-due-to-age Locked due to a period of inactivity. Please open new issues or PRs if more discussion is needed. label Feb 22, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 22, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
frozen-due-to-age Locked due to a period of inactivity. Please open new issues or PRs if more discussion is needed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants