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

[15.0][MIG] sale_order_secondary_unit: Migration to 15.0 #2057

Merged
merged 20 commits into from
Aug 16, 2022

Conversation

CarlosRoca13
Copy link
Contributor

cc @Tecnativa TT36530

superseded from #1845

please @sergio-teruel @victoralmau review this

@victoralmau
Copy link
Member

Please check CI errors.

@FedericoLaTorre
Copy link

Product matrix stop adding quantities after installing this module.

@CarlosRoca13 CarlosRoca13 force-pushed the 15.0-MIG-sale_order_secondary_unit branch 2 times, most recently from 8f05cff to e5ca0e4 Compare August 9, 2022 10:14
@CarlosRoca13 CarlosRoca13 force-pushed the 15.0-MIG-sale_order_secondary_unit branch from f36439e to b4ce7d9 Compare August 9, 2022 11:10
@sergio-teruel
Copy link
Contributor

@FedericoLaTorre Fixed! Can you review?

@FedericoLaTorre
Copy link

Works great now! I couldn't find any issues.

@pedrobaeza
Copy link
Member

Please approve it through regular button, @FedericoLaTorre

@pedrobaeza
Copy link
Member

/ocabot migration sale_order_secondary_unit

Merging due to the implicit approvals:

/ocabot merge nobump

@OCA-git-bot OCA-git-bot added this to the 15.0 milestone Aug 16, 2022
@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 15.0-ocabot-merge-pr-2057-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot
Copy link
Contributor

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

pilarvargas-tecnativa pushed a commit to Tecnativa/sale-workflow that referenced this pull request Feb 7, 2023
Signed-off-by pedrobaeza
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.