Skip to content

[IMP]: adapt tours #4485

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

Draft
wants to merge 405 commits into
base: master-mysterious-egg
Choose a base branch
from

Conversation

shsa-odoo
Copy link

drop_404_ir_attachment_url

agau-odoo and others added 30 commits April 18, 2025 11:19
This commit addresses a race condition in the `onEditPage` method of
the `WebsiteBuilder` component. The issue occurred when asynchronous
operations (like iframe loading and asset bundle loading) were not
completed before proceeding with the editing flow, leading to
unpredictable behavior.

Changes:
- Added `await this.iframeLoaded` in `onEditPage` to ensure the iframe
is fully loaded before loading assets or enabling editing.
- Set the `is-ready` attribute on the iframe element in `onIframeLoad`
to mark it as ready once the iframe content is loaded.

These changes ensure that all dependencies are properly initialized
before enabling the editing mode, resolving the race condition.
paru-odoo and others added 28 commits April 22, 2025 11:16
Now handling the Google Maps API in the iframe with an interaction, we
can't test it anymore since the tests don't have access to the iframe
and can't make overrides there, and therefore doesn't load the
interaction.
@robodoo
Copy link

robodoo commented Apr 25, 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 branch from e24556e to 897f4ab Compare April 29, 2025 18:30
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.