-
-
Notifications
You must be signed in to change notification settings - Fork 750
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
[16.0][MIG] purchase_line_procurement_group: Migration to 16.0 #1806
[16.0][MIG] purchase_line_procurement_group: Migration to 16.0 #1806
Conversation
This reverts commit aa4bd6c.
This reverts commit bd75195.
Currently translated at 100.0% (4 of 4 strings) Translation: purchase-workflow-12.0/purchase-workflow-12.0-purchase_line_procurement_group Translate-URL: https://translation.odoo-community.org/projects/purchase-workflow-12-0/purchase-workflow-12-0-purchase_line_procurement_group/zh_CN/
and a service was included. The service does not create a stock move.
Currently translated at 100.0% (4 of 4 strings) Translation: purchase-workflow-12.0/purchase-workflow-12.0-purchase_line_procurement_group Translate-URL: https://translation.odoo-community.org/projects/purchase-workflow-12-0/purchase-workflow-12-0-purchase_line_procurement_group/fr/
This reverts commit e2cdc23405318a2af0bc08405013917819bfd4d6.
…t procurement group This fix restore the behaviour found in the 12.0 implementation of this module. Without it, an existing purchase line with a procurement group can be merged with a procurement without group, while the reverse is not true.
@lehoangan, I think you should add yourself in the contributor, |
I addes it anlh@trobz.com |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code review
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This PR has the |
/ocabot merge nobump |
This PR looks fantastic, let's merge it! |
Congratulations, your PR was merged at a5aa660. Thanks a lot for contributing to OCA. ❤️ |
No description provided.