Skip to content

Conversation

@mart-e
Copy link
Contributor

@mart-e mart-e commented Feb 14, 2022

When a user clicks on the link of an alternate page in the version or
language switcher, we now check if the page referenced by the target URL
exists or not. If not, we generate a series of fallback URLs from the
target URL and check whether the targeted resource exists or not, until
we read the root of the documentation. As soon as we find a valid URL,
we redirect the user to it.

This is inspired by the behaviour of docs.python.org's version and
language switchers.

task-2534669

@robodoo
Copy link
Collaborator

robodoo commented Feb 14, 2022

@mart-e mart-e force-pushed the 13.0-fallback-urls-switcher-mat branch 4 times, most recently from d258bea to 70fe30d Compare February 14, 2022 14:53
@robodoo
Copy link
Collaborator

robodoo commented Apr 22, 2022

Sorry, I didn't know about this PR and had to retrieve its information, you may have to re-approve it.

@robodoo
Copy link
Collaborator

robodoo commented Apr 22, 2022

@mart-e mart-e force-pushed the 13.0-fallback-urls-switcher-mat branch 2 times, most recently from 96c1bb4 to 9dc83a6 Compare April 22, 2022 09:54
@mart-e mart-e force-pushed the 13.0-fallback-urls-switcher-mat branch 2 times, most recently from 689475c to 80f8f6b Compare May 17, 2022 15:07
@mart-e mart-e marked this pull request as ready for review May 17, 2022 15:08
@mart-e mart-e changed the title [WIP] odoo_theme: better management of non-existant pages [IMP] odoo_theme: better management of non-existant pages May 17, 2022
@C3POdoo C3POdoo requested review from a team May 17, 2022 15:09
@AntoineVDV AntoineVDV force-pushed the 13.0-fallback-urls-switcher-mat branch from 80f8f6b to fb87d9e Compare May 19, 2022 12:16
@AntoineVDV AntoineVDV changed the title [IMP] odoo_theme: better management of non-existant pages [IMP] odoo_theme: introduce fallback URLs for the page switchers May 19, 2022
@AntoineVDV AntoineVDV force-pushed the 13.0-fallback-urls-switcher-mat branch from fb87d9e to d048576 Compare May 19, 2022 12:17
@AntoineVDV AntoineVDV removed the request for review from a team May 19, 2022 12:19
@AntoineVDV AntoineVDV force-pushed the 13.0-fallback-urls-switcher-mat branch 3 times, most recently from 2c88104 to b720612 Compare May 19, 2022 14:21
When a user clicks on the link of an alternate page in the version or
language switcher, we now check if the page referenced by the target URL
exists or not. If not, we generate a series of fallback URLs from the
target URL and check whether the targeted resource exists or not, until
we read the root of the documentation. As soon as we find a valid URL,
we redirect the user to it.

This is inspired by the behaviour of docs.python.org's version and
language switchers.

task-2534669

Co-authored-by: Antoine Vandevenne (anv) <[email protected]>
@AntoineVDV AntoineVDV force-pushed the 13.0-fallback-urls-switcher-mat branch from b720612 to 074c91f Compare May 19, 2022 14:24
Copy link
Collaborator

@AntoineVDV AntoineVDV left a comment

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 May 19, 2022
When a user clicks on the link of an alternate page in the version or
language switcher, we now check if the page referenced by the target URL
exists or not. If not, we generate a series of fallback URLs from the
target URL and check whether the targeted resource exists or not, until
we read the root of the documentation. As soon as we find a valid URL,
we redirect the user to it.

This is inspired by the behaviour of docs.python.org's version and
language switchers.

task-2534669

closes #1588

Signed-off-by: Antoine Vandevenne (anv) <[email protected]>
Co-authored-by: Antoine Vandevenne (anv) <[email protected]>
@robodoo robodoo closed this May 19, 2022
@robodoo robodoo temporarily deployed to merge May 19, 2022 16:16 Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants