Skip to content

[REF] runbot: prepare terrain for owl capabilities #1090

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

Open
wants to merge 7 commits into
base: 18.0
Choose a base branch
from

Conversation

Williambraecky
Copy link
Contributor

  • Reorganize static folder
  • Fix diff_match_patch export which was borked
  • Vendor owl.js and expose through odoo module
  • Add a way to render owl components on the frontend
  • Add a way to add interaction through colibri framework
  • Rewrite runbot.js using colibri
  • Remove direct rels to assets in layout template

- Move libs from ./static/src/libs to ./static/libs
- Move img from ./static/src/img to ./static/img
- Remove the ./js subfolder of src
- Remove empty log_display.js file
- Remove the unused runbot.assets_frontend bundle
The lib exports its data through the global object, an odoo module then
re-exports the module to make it available to other odoo modules.
Adds the minimal requirements for owl and xml file based templating for
the frontend.
Copies the colibri + interaction framework from `web` module.
We also bootstrap interactions to be used in the frontend.
Rewrites runbot.js in modern js with the new Interaction framework.
Instead of hardcoding rels, move assets to the bundle.
@C3POdoo C3POdoo requested a review from a team March 10, 2025 15:43
@Williambraecky
Copy link
Contributor Author

Diff is big but this is necessary if we want modern javascript for future improvements/refactoring.

JQuery is only required by the stats page, which #1035 already covers.

@Williambraecky Williambraecky force-pushed the 18.0-vendor-owl-wbr branch 3 times, most recently from 4adff2f to cc2e3e0 Compare March 24, 2025 08:22
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.

1 participant