Skip to content

[#73926] Add canonical meta tags to project & WP show pages#22813

Draft
thykel wants to merge 1 commit intodevfrom
feat/73926-meta_tags_for_semantic_ids
Draft

[#73926] Add canonical meta tags to project & WP show pages#22813
thykel wants to merge 1 commit intodevfrom
feat/73926-meta_tags_for_semantic_ids

Conversation

@thykel
Copy link
Copy Markdown
Contributor

@thykel thykel commented Apr 17, 2026

Ticket

https://community.openproject.org/projects/stream-jira-exit/work_packages/73926/activity

Merge Pre-requirements

What are you trying to accomplish?

We have been implementing support for historical project URLs as well as semantic identifiers. A side effect of this is the fact that OP may now generate multiple links that lead to the exact same resource.

We do not want to confuse crawlers, so let's group those multiple versions together via a single canonical meta tag.

Screenshots

Screenshot 2026-04-17 at 14 54 14 Screenshot 2026-04-17 at 14 54 45

What approach did you choose and why?

The tag is present on top of every project/WP show page, no matter which URL was used to get to it or which tab is open in the UI.

The unified canonical format is /projects/:numeric_db_id/work_packages/:numeric_db_id.

You may consult the work package to see the reasoning behind this specific format.

Merge checklist

  • Added/updated tests
  • Added/updated documentation in Lookbook (patterns, previews, etc)
  • Tested major browsers (Chrome, Firefox, Edge, ...)

@thykel thykel changed the title [#73926] Add canonical meta tags to Project/WP show pages [#73926] Add canonical meta tags to project & WP show pages Apr 17, 2026
@thykel thykel force-pushed the feat/73926-meta_tags_for_semantic_ids branch from 16b5fad to 55becdb Compare April 17, 2026 13:38
@thykel thykel force-pushed the feat/73926-meta_tags_for_semantic_ids branch from 55becdb to 637d986 Compare April 17, 2026 13:42
@thykel thykel marked this pull request as ready for review April 17, 2026 13:42
@thykel thykel requested review from akabiru and judithroth April 17, 2026 13:43
@thykel thykel marked this pull request as draft April 17, 2026 13:51
@akabiru
Copy link
Copy Markdown
Member

akabiru commented Apr 17, 2026

@thykel worth adding the feature flag to this one?

@thykel thykel marked this pull request as ready for review April 20, 2026 04:32
@thykel
Copy link
Copy Markdown
Contributor Author

thykel commented Apr 20, 2026

@thykel worth adding the feature flag to this one?

Actually, I don't think so. When it comes to crawler optimization, we want to have these exposed as early and as consistently as possible.

There's already value in this even for the classic mode, as you can already have multiple URLs pointing to the same resource (slug-based vs. ID-based).

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 21, 2026

Deploying openproject with PullPreview

Field Value
Latest commit 637d986
Job deploy
Status ✅ Deploy successful
Preview URL https://pr-22813-73926-meta-tags-fo-ip-91-98-85-142.my.opf.run:443

View logs

@thykel thykel marked this pull request as draft April 21, 2026 08:15
Base automatically changed from feature/73756-adapt-routes-for-project-based-semantic-work-package-identifiers to dev April 21, 2026 10:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants