-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
[18.0][ADD] sale_order_line_client_order_ref #4052
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
[18.0][ADD] sale_order_line_client_order_ref #4052
Conversation
|
Tests CI will be fixed after #4053 is merged. |
6a640b9 to
25aef53
Compare
yostashiro
left a comment
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.
@AungKoKoLin1997 Can you please add an option (company-level config) to show Customer Order Ref in PDF reports (quotation and invoice)?
25aef53 to
689b1b5
Compare
@yostashiro Done! |
sale_order_line_client_order_ref/reports/report_invoice_document.xml
Outdated
Show resolved
Hide resolved
sale_order_line_client_order_ref/reports/report_saleorder_document.xml
Outdated
Show resolved
Hide resolved
689b1b5 to
cabfb1f
Compare
yostashiro
left a comment
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.
Please apply the same idea to the account config side.
sale_order_line_client_order_ref/views/res_config_settings_views.xml
Outdated
Show resolved
Hide resolved
8f51563 to
3e03c00
Compare
yostashiro
left a comment
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 and functional test. LGTM apart from the CI error which should be fixed by #4053.
3e03c00 to
a221fed
Compare
@AungKoKoLin1997 And why not creating several sale orders ? |
@rousseldenis |
|
/ocabot merge nobump |
|
Hey, thanks for contributing! Proceeding to merge this for you. |
|
Congratulations, your PR was merged at 9d7309d. Thanks a lot for contributing to OCA. ❤️ |
This module adds Customer Order Number to sale order line and propose the field to invoice line.
This can be used in case there are situations where a sales order contains multiple customer orders.
@qrtl QT6179