Skip to content
This repository has been archived by the owner on Feb 22, 2023. It is now read-only.

All pages should output a canonical URL tag #812

Closed
jonoalderson opened this issue Feb 4, 2022 · 3 comments
Closed

All pages should output a canonical URL tag #812

jonoalderson opened this issue Feb 4, 2022 · 3 comments
Labels
💻 aspect: code Concerns the software code in the repository 🛠 goal: fix Bug fix 🟧 priority: high Stalls work on the project or its dependents 🚧 status: blocked Blocked & therefore, not ready for work
Projects

Comments

@jonoalderson
Copy link

Description

All pages* on the openverse site must output a canonical URL tag, which correctly references the 'definitive' URL for that page.
Currently, many pages omit this property. E.g., https://wordpress.org/openverse/meta-search, https://ja.wordpress.org/openverse/extension, https://wordpress.org/openverse/search-help

*Where 'page' excludes error states and search-results.

Note that this is not a definitive list of URLs, and that logic should be applied globally to all page and template types, rather than URL-by-URL.

Correcting this is critical, as part of our ongoing SEO firefighting efforts across the wp.org ecosystem.

Reproduction

  1. Visit one of the example pages above.
  2. View the HTML source code.
  3. See that there is no canonical URL tag.

Expectation

All pages* should output a (correct) canonical URL tag.

@jonoalderson jonoalderson added 💻 aspect: code Concerns the software code in the repository 🚦 status: awaiting triage Has not been triaged & therefore, not ready for work 🛠 goal: fix Bug fix 🟧 priority: high Stalls work on the project or its dependents labels Feb 4, 2022
@zackkrida zackkrida transferred this issue from WordPress/openverse Feb 4, 2022
@jonoalderson
Copy link
Author

Also note that URLs like https://wordpress.org/openverse/image/251f17bc-37a4-4cdd-b054-ceb0d52b445e/?referrer=creativecommons.org currently output an incorrect canonical URL value (of https://wordpress.org/openverse/), which is resulting in the tag being completely ignored.

@sarayourfriend
Copy link
Contributor

This is another iframe problem that will be hard/tedius to solve.

Aside from all the usability issues with the iframe, it will force us to duplicate all SEO work in both the actual site and in the wporg theme container.

Still working on figuring out what the path forward is with the iframe. Waiting on feedback from the infrastructure side of things still on whether it's a possibility.

@krysal krysal removed the 🚦 status: awaiting triage Has not been triaged & therefore, not ready for work label Mar 7, 2022
@sarayourfriend sarayourfriend added the 🚧 status: blocked Blocked & therefore, not ready for work label Apr 25, 2022
@krysal krysal moved this from Backlog to Blocked in Openverse Jun 21, 2022
@obulat obulat added this to the SEO improvements milestone Nov 20, 2022
@zackkrida
Copy link
Member

Fixed by launching to openverse.org. Canonical links are currently relative and will be made absolute in this PR: #2157

Openverse automation moved this from Blocked to Done! Feb 8, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
💻 aspect: code Concerns the software code in the repository 🛠 goal: fix Bug fix 🟧 priority: high Stalls work on the project or its dependents 🚧 status: blocked Blocked & therefore, not ready for work
Projects
No open projects
Openverse
  
Done!
Development

No branches or pull requests

5 participants