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

wip #4194

Merged
merged 1 commit into from
Mar 27, 2025
Merged

wip #4194

merged 1 commit into from
Mar 27, 2025

Conversation

ged-odoo
Copy link

No description provided.

@robodoo
Copy link

robodoo commented Mar 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.

@ged-odoo ged-odoo force-pushed the master-mysterious-egg-ged branch 2 times, most recently from ca5ac98 to b29f16a Compare March 14, 2025 13:55
if (asyncActions.length) {
onWillStart(async function () {
await Promise.all(asyncActions.map((obj) => obj.action.prepare(obj.descr)));
comp.env.editorBus.trigger("DOM_UPDATED");

Choose a reason for hiding this comment

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

??? why ?
We maybe need useDomState with onWillStart ?(beso had an issue like that)

@ged-odoo ged-odoo force-pushed the master-mysterious-egg-ged branch 4 times, most recently from 66cac0c to 87d58db Compare March 18, 2025 13:07
@FrancoisGe FrancoisGe force-pushed the master-mysterious-egg branch from 3ee87bf to 98b990a Compare March 19, 2025 09:22
@ged-odoo ged-odoo force-pushed the master-mysterious-egg-ged branch 2 times, most recently from 2d79cdf to 3fbef77 Compare March 19, 2025 13:12
@ged-odoo ged-odoo force-pushed the master-mysterious-egg-ged branch from 3fbef77 to f0c680f Compare March 26, 2025 13:23
@ged-odoo ged-odoo force-pushed the master-mysterious-egg-ged branch from f0c680f to 67ef856 Compare March 27, 2025 09:36
@robinlej robinlej merged commit 2ca1182 into master-mysterious-egg Mar 27, 2025
@robinlej robinlej deleted the master-mysterious-egg-ged branch March 27, 2025 10:26
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