Skip to content

[IMP] website: support unslugified url for redirect#277992

Closed
fw-bot wants to merge 1 commit into
odoo:saas-19.3from
odoo-dev:saas-19.3-19.0-redirect-slug-jke-549297-fw
Closed

[IMP] website: support unslugified url for redirect#277992
fw-bot wants to merge 1 commit into
odoo:saas-19.3from
odoo-dev:saas-19.3-19.0-redirect-slug-jke-549297-fw

Conversation

@fw-bot

@fw-bot fw-bot commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

In some cases, you want to redirect a record with ModelConverter, whatever the slug value.

E.g. /shop/old-name-1 => /shop/alt-product-10
/shop/new-name-1 => /shop/alt-product-10
/fr/shop/nom-1 => /shop/alternatif-product-10
/de/shop/produktname-1 => /de/shop/produktname-10

In this case, adding only one redirect /shop/1 => /shop/10 covers the need to support all the translated slugs and the old name that we remember.

On odoo.com we have this need e.g. when we archive a Job Position, we create a redirect, but in some cases the job position is translated or has been renamed and we don't remember all the old urls. With this change, we will be able to redirect all old urls, translated urls, ... with only one redirect.

/jobs/10 -> /explore-more-opportunities-with-us

task-6391567

Forward-Port-Of: #276515

In some cases, you want to redirect a record with ModelConverter,
whatever the slug value.

E.g. /shop/old-name-1 => /shop/alt-product-10
     /shop/new-name-1 => /shop/alt-product-10
     /fr/shop/nom-1 => /shop/alternatif-product-10
     /de/shop/produktname-1 => /de/shop/produktname-10

In this case, adding only one redirect /shop/1 => /shop/10 covers the
need to support all the translated slugs and the old name that we
remember.

On odoo.com we have this need e.g. when we archive a Job Position, we
create a redirect, but in some cases the job position is translated or
has been renamed and we don't remember all the old urls. With this
change, we will be able to redirect all old urls, translated urls, ...
with only one redirect.

/jobs/10 -> /explore-more-opportunities-with-us

task-6391567

X-original-commit: d3bf488
@robodoo

robodoo commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Pull request status dashboard

@robodoo robodoo added the forwardport This PR was created by @fw-bot label Jul 23, 2026
@fw-bot

fw-bot commented Jul 23, 2026

Copy link
Copy Markdown
Contributor Author

This PR targets saas-19.3 and is part of the forward-port chain. Further PRs will be created up to master.

More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port

@fw-bot

fw-bot commented Jul 23, 2026

Copy link
Copy Markdown
Contributor Author

@JKE-be @h4818 the next pull request (#278045) is in conflict. You can merge the chain up to here by saying

@robodoo r+

More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port

@h4818 h4818 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

robodoo pushed a commit that referenced this pull request Jul 23, 2026
In some cases, you want to redirect a record with ModelConverter,
whatever the slug value.

E.g. /shop/old-name-1 => /shop/alt-product-10
     /shop/new-name-1 => /shop/alt-product-10
     /fr/shop/nom-1 => /shop/alternatif-product-10
     /de/shop/produktname-1 => /de/shop/produktname-10

In this case, adding only one redirect /shop/1 => /shop/10 covers the
need to support all the translated slugs and the old name that we
remember.

On odoo.com we have this need e.g. when we archive a Job Position, we
create a redirect, but in some cases the job position is translated or
has been renamed and we don't remember all the old urls. With this
change, we will be able to redirect all old urls, translated urls, ...
with only one redirect.

/jobs/10 -> /explore-more-opportunities-with-us

task-6391567

closes #277992

X-original-commit: d3bf488
Signed-off-by: Habib Ayob (ayh) <ayh@odoo.com>
@robodoo robodoo closed this Jul 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

forwardport This PR was created by @fw-bot

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants