-
Notifications
You must be signed in to change notification settings - Fork 6.6k
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
fix: payment term when creating PO from SO #41376
base: develop
Are you sure you want to change the base?
Conversation
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.
Thanks for the quick fix. Added a minor comment.
@@ -1239,11 +1239,18 @@ def set_missing_values(source, target): | |||
target.discount_amount = 0.0 | |||
target.inter_company_order_reference = "" | |||
target.shipping_rule = "" | |||
target.tc_name = "" | |||
target.terms = "" | |||
target.payment_terms_template = "" |
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.
I think the payment_schedule
should be cleared. Only clearing the payment_terms_template
didn't work for me. The payment_schedule
still got copied.
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.
Only works properly in version 15. I think the issue will be in version 14, but I fixed 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.
Thanks!
Version 15 and 14,
fixes: #41369
Before:
before_so_to_po.mp4
After:
after_so_to_po.mp4