Skip to content

improve save mechanism #4681

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

Closed

Conversation

loco-odoo
Copy link

No description provided.

@robodoo
Copy link

robodoo commented May 14, 2025

This PR targets the un-managed branch odoo-dev/odoo:master-mysterious-egg, it needs to be retargeted before it can be merged.

@loco-odoo loco-odoo force-pushed the master-mysterious-egg-loco-2 branch 3 times, most recently from c2bc17a to dda32d5 Compare May 19, 2025 07:50
@FrancoisGe FrancoisGe force-pushed the master-mysterious-egg branch from 5b240d1 to 558b8e7 Compare May 20, 2025 05:41
@loco-odoo loco-odoo force-pushed the master-mysterious-egg-loco-2 branch 2 times, most recently from 1afd066 to 16e819d Compare May 20, 2025 08:43
const resultSave = [];
onRpc("/web_editor/field/translation/update", async (data) => {
const { params } = await data.json();
resultSave.push(params.translations.fr_BE);

Choose a reason for hiding this comment

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

why not use expect.step ?

Choose a reason for hiding this comment

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

you want to check if you call one or two time "web_editor/field/translation/update"
Maybe add a test where you need to do 2 call and one test with 1 call
PS we need to move python code to website or html builder ?

@loco-odoo loco-odoo force-pushed the master-mysterious-egg-loco-2 branch from 16e819d to 88ce30b Compare May 20, 2025 11:12
@ged-odoo ged-odoo force-pushed the master-mysterious-egg branch from ea446c5 to f76e670 Compare May 21, 2025 08:21
ged-odoo and others added 2 commits May 21, 2025 10:40
…itor

This commit introduces a full rewrite of the website builder to leverage
the OWL framework and the new HTML editor. The goal is to modernize the
codebase, improve maintainability, and provide a more reactive and
modular architecture.

A major part of this refactoring is the complete rethinking of how
options are defined. They are now implemented as OWL components or simple
template-driven views, and registered as plugins. The system introduces
two distinct concepts: *options*, which handle the user interface, and
*actions*, which encapsulate the actual DOM or HTML updates.

The builder code is now split into two addons:
- `html_builder`: contains a shared foundation for building HTML UIs,
  designed to be reusable across apps, including mass mailing.
- `website`: contains all website-specific logic and components.

The legacy builder code has not been removed yet, as parts of it are
still in use, most notably in the mass mailing application. It will be
removed in the near future.

Many thanks to everyone who contributed to this project, whether through
code, testing, reviews, design, or support—your efforts made this major
milestone possible. Also, this sentence was totally not generated by a AI
language model.

Co-authored-by: aans-odoo <[email protected]>
Co-authored-by: Alessandro Lupo <[email protected]>
Co-authored-by: Alice Gaudon <[email protected]>
Co-authored-by: Antoine Guenet <[email protected]>
Co-authored-by: Augustin (duau) <[email protected]>
Co-authored-by: Benoit Socias <[email protected]>
Co-authored-by: chdh-odoo <[email protected]>
Co-authored-by: Davide Bonetto <[email protected]>
Co-authored-by: Denis Rygaert <[email protected]>
Co-authored-by: divy-odoo <[email protected]>
Co-authored-by: emge-odoo <[email protected]>
Co-authored-by: fdardenne <[email protected]>
Co-authored-by: FrancoisGe <[email protected]>
Co-authored-by: Géry Debongnie <[email protected]>
Co-authored-by: Guillaume Jacquet <[email protected]>
Co-authored-by: Jinjiu Liu <[email protected]>
Co-authored-by: ksbh-odoo <[email protected]>
Co-authored-by: Louis (loco) <[email protected]>
Co-authored-by: Mohammed Shekha <[email protected]>
Co-authored-by: Nicolas Bayet <[email protected]>
Co-authored-by: panv-odoo <[email protected]>
Co-authored-by: paru-odoo <[email protected]>
Co-authored-by: Rahil Ghanchi <[email protected]>
Co-authored-by: Robin Lejeune (role) <[email protected]>
Co-authored-by: Rodolpho Lima <[email protected]>
Co-authored-by: Sanjay Sharma <[email protected]>
Co-authored-by: Sébastien Blondiau <[email protected]>
Co-authored-by: Sébastien Geelen <[email protected]>
Co-authored-by: Serhii Rubanskyi - seru <[email protected]>
Co-authored-by: Soukéina Bojabza <[email protected]>
Co-authored-by: Subhadeep <[email protected]>
Co-authored-by: visp-odoo <[email protected]>
@ged-odoo ged-odoo force-pushed the master-mysterious-egg branch 4 times, most recently from edff5d7 to d31ecfa Compare May 21, 2025 13:57
@FrancoisGe FrancoisGe force-pushed the master-mysterious-egg branch 3 times, most recently from 314790f to 48edd7d Compare May 22, 2025 06:44
@loco-odoo loco-odoo force-pushed the master-mysterious-egg-loco-2 branch from 88ce30b to 4eb9e12 Compare May 22, 2025 07:01
@loco-odoo loco-odoo changed the base branch from master-mysterious-egg to master-mysterious-egg-next May 22, 2025 07:01
@ged-odoo ged-odoo force-pushed the master-mysterious-egg-next branch from 1d49bd6 to 6e36b56 Compare May 22, 2025 07:24
@loco-odoo loco-odoo force-pushed the master-mysterious-egg-loco-2 branch from 4eb9e12 to 292f6f7 Compare May 22, 2025 07:31
@ged-odoo ged-odoo force-pushed the master-mysterious-egg-next branch 6 times, most recently from 4f4c0ae to 66b2367 Compare May 26, 2025 13:45
@ged-odoo ged-odoo force-pushed the master-mysterious-egg-next branch 7 times, most recently from 1edbd4e to 4392eae Compare May 29, 2025 18:10
@fw-bot fw-bot deleted the branch master-mysterious-egg-next June 6, 2025 02:02
@fw-bot fw-bot closed this Jun 6, 2025
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.

5 participants