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

Obtain props from function for attached Owl components #3463

Draft
wants to merge 12 commits into
base: master-public-widgets
Choose a base branch
from

Conversation

bso-odoo
Copy link

No description provided.

@robodoo
Copy link

robodoo commented Oct 29, 2024

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

@ged-odoo ged-odoo force-pushed the master-public-widgets-ged branch 2 times, most recently from 86f4063 to 6bbd579 Compare October 30, 2024 15:51
@ged-odoo ged-odoo force-pushed the master-public-widgets-ged branch from 6bbd579 to 21714ef Compare October 31, 2024 08:28
@bso-odoo bso-odoo force-pushed the master-public-widgets-bso branch from 0e5c60d to c1afdba Compare October 31, 2024 09:27
@ged-odoo ged-odoo force-pushed the master-public-widgets-ged branch 2 times, most recently from cc9630a to 9e0215d Compare October 31, 2024 13:52
Base automatically changed from master-public-widgets-ged to master-public-widgets October 31, 2024 13:55
@bso-odoo bso-odoo force-pushed the master-public-widgets-bso branch 5 times, most recently from c876103 to 4417ade Compare November 4, 2024 08:11
return;
}
// Escape special characters to make the jQuery selector to work.
hash = '#' + $.escapeSelector(hash.substring(1));

Choose a reason for hiding this comment

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

jquery

Copy link
Author

Choose a reason for hiding this comment

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

Added TODO comment about getting rid of JQ there: that escaping is non-trivial, and it might have to be added in a place like dom.js - but it is surprisingly only used in that one location.

@bso-odoo bso-odoo force-pushed the master-public-widgets-bso branch 3 times, most recently from fa11dd4 to 4dc83ce Compare November 4, 2024 12:20
@bso-odoo bso-odoo force-pushed the master-public-widgets-bso branch 2 times, most recently from d23d65f to bfdfbd2 Compare November 5, 2024 09:50
@bso-odoo bso-odoo force-pushed the master-public-widgets-bso branch 6 times, most recently from 06148f2 to 03a4927 Compare November 6, 2024 10:09
@ged-odoo ged-odoo force-pushed the master-public-widgets branch from 2cc48c9 to b746ee1 Compare November 6, 2024 10:11
@bso-odoo bso-odoo force-pushed the master-public-widgets-bso branch from 03a4927 to 1a54cdc Compare November 6, 2024 10:33
@ged-odoo ged-odoo force-pushed the master-public-widgets branch from 62d6dc8 to bf2b6b1 Compare November 6, 2024 10:54
@ged-odoo ged-odoo force-pushed the master-public-widgets branch 2 times, most recently from 1d47261 to d11e6df Compare November 18, 2024 08:25
@ged-odoo ged-odoo force-pushed the master-public-widgets branch from f41bf58 to 7fd0d01 Compare December 2, 2024 08:10
@ged-odoo ged-odoo force-pushed the master-public-widgets branch from 4408e26 to daa5c3e Compare December 17, 2024 10:35
@ged-odoo ged-odoo force-pushed the master-public-widgets branch 4 times, most recently from bd90205 to 3b99803 Compare January 13, 2025 10:29
@ged-odoo ged-odoo force-pushed the master-public-widgets branch from 4775765 to 163eaa9 Compare January 15, 2025 14:19
@robinlej robinlej force-pushed the master-public-widgets branch 3 times, most recently from 6910b8e to a06a3d5 Compare January 17, 2025 15:55
@ged-odoo ged-odoo force-pushed the master-public-widgets branch from 5d1b4c3 to 72f8a28 Compare January 20, 2025 08:24
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