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

[FW][IMP] howtos/website_themes: Rework of the Layout page #12475

Closed

Conversation

fw-bot
Copy link
Collaborator

@fw-bot fw-bot commented Mar 14, 2025

This PR updates current "Layout" page:

  • New anchors ;
  • Content update.

Task-4316611

Forward-Port-Of: #12220

@robodoo
Copy link
Collaborator

robodoo commented Mar 14, 2025

Pull request status dashboard

@fw-bot
Copy link
Collaborator Author

fw-bot commented Mar 14, 2025

This PR targets 17.0 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
Copy link
Collaborator Author

fw-bot commented Mar 14, 2025

@bram-odoo ci/documentation failed on this forward-port PR

[IMP] Website Themes - Layout page

[FIX] Corrections for ci

[FIX] Ci failure

[IMP] Small corrections

- Reorder the Xpath examples.
- Add an explanation for the move directive

[ADD] Website Themes - Media section

[IMP] Website Themes - Layout section modifications

[IMP] Anchors for CEDE's references (Tutorials)

[IMP] Line width + visibility conditions alert type

[IMP] Website Themes - Layout : Anchors correction

Revert "[ADD] Website Themes - Media section"

This reverts commit 4567cfe.

[FIX] Update anchors pattern

[FIX] Media company logo reference

X-original-commit: 53b2db4
@bram-odoo bram-odoo force-pushed the 17.0-16.0-website-howto-theming-layout-bram-419521-fw branch from 39d6f02 to c5952bb Compare March 15, 2025 06:13
@fw-bot
Copy link
Collaborator Author

fw-bot commented Mar 15, 2025

@bram-odoo this PR was modified / updated and has become a normal PR. It must be merged directly.

@C3POdoo C3POdoo requested a review from a team March 15, 2025 06:15
@bram-odoo
Copy link
Contributor

@robodoo r+

@bram-odoo
Copy link
Contributor

bram-odoo commented Mar 15, 2025

Hi @AntoineVDV, I had to rebase this branch in order to have the media section created by @karlosvalverde (The PR related to the media section was totally deployed (also in 17) at that moment, I think. But now I get an error on some FW PR talking about:

This certificate is not valid, its validity has expired.

Do you know what's going on and how to fix it?

@bram-odoo bram-odoo requested a review from AntoineVDV March 15, 2025 06:17
robodoo pushed a commit that referenced this pull request Mar 15, 2025
[IMP] Website Themes - Layout page

[FIX] Corrections for ci

[FIX] Ci failure

[IMP] Small corrections

- Reorder the Xpath examples.
- Add an explanation for the move directive

[ADD] Website Themes - Media section

[IMP] Website Themes - Layout section modifications

[IMP] Anchors for CEDE's references (Tutorials)

[IMP] Line width + visibility conditions alert type

[IMP] Website Themes - Layout : Anchors correction

Revert "[ADD] Website Themes - Media section"

This reverts commit 4567cfe.

[FIX] Update anchors pattern

[FIX] Media company logo reference

closes #12475

X-original-commit: 53b2db4
Signed-off-by: Brandon Mercier (bram) <[email protected]>
@robodoo robodoo closed this Mar 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants