-
-
Notifications
You must be signed in to change notification settings - Fork 746
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
[17.0][MIG] purchase_sale_link_by_origin: Migration to 17.0 #2266
[17.0][MIG] purchase_sale_link_by_origin: Migration to 17.0 #2266
Conversation
Currently translated at 100.0% (2 of 2 strings) Translation: purchase-workflow-16.0/purchase-workflow-16.0-purchase_sale_link_by_origin Translate-URL: https://translation.odoo-community.org/projects/purchase-workflow-16-0/purchase-workflow-16-0-purchase_sale_link_by_origin/it/
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
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.
cosmetic changes, pre-approving, since it doesn't affect the module itself
By default since [Odoo | ||
14.0](https://github.com/odoo/odoo/commit/5a1645a8f8f3560eb778da90b6160b322ce0722e), | ||
PO and SO are linked by their order lines. | ||
|
||
This module also link them by the PO's Origin field, to cover more | ||
cases. For example: - If a user cancels a PO, by default the link would | ||
have been broken; now it won't; - Or if a user manually defines or | ||
updates the Origin field of a PO, it will be taken into account. |
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.
formatting is broken here
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.
4e6f92a
to
9fecad7
Compare
Updated |
/ocabot merge nobump |
On my way to merge this fine PR! |
Congratulations, your PR was merged at 4535674. Thanks a lot for contributing to OCA. ❤️ |
No description provided.