Skip to content

[18.0][MIG] pos_divide_order_summary: Migration to 18.0 #1351

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

Merged
merged 3 commits into from
Apr 9, 2025

Conversation

trisdoan
Copy link
Contributor

@trisdoan trisdoan commented Apr 1, 2025

This changes

  • there was a refactor https://github.com/odoo/odoo/pull/183299, which impacts template
  • test file structure was also changed in https://github.com/odoo/odoo/pull/157180

Result

2025-04-01_11-09

henrybackman and others added 3 commits March 28, 2025 15:33
Divides the order summary into two parts allowing adding content to the
left-hand side of the order summary.
Other modules depend on this to be shown correctly. For example:
- pos_display_order_number
- pos_display_total_quantity
Copy link

@sebduch73 sebduch73 left a comment

Choose a reason for hiding this comment

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

I tested the module and everything is ok. Can you push the PR?

@trisdoan
Copy link
Contributor Author

trisdoan commented Apr 9, 2025

Hello @henrybackman @ivantodorovich , how does it look?

Copy link
Contributor

@ivantodorovich ivantodorovich left a comment

Choose a reason for hiding this comment

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

/ocabot migration pos_divide_order_summary

@OCA-git-bot OCA-git-bot added this to the 18.0 milestone Apr 9, 2025
@OCA-git-bot OCA-git-bot mentioned this pull request Apr 9, 2025
14 tasks
@ivantodorovich
Copy link
Contributor

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 18.0-ocabot-merge-pr-1351-by-ivantodorovich-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit cb2ed4f into OCA:18.0 Apr 9, 2025
7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at f5744d9. Thanks a lot for contributing to OCA. ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants