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

fix: payment term when creating PO from SO #41376

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

Nihantra-Patel
Copy link
Contributor

Version 15 and 14,

fixes: #41369

Before:

before_so_to_po.mp4

After:

after_so_to_po.mp4

@github-actions github-actions bot added the needs-tests This PR needs automated unit-tests. label May 8, 2024
Copy link
Contributor

@IMS94 IMS94 left a 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 = ""
Copy link
Contributor

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.

Copy link
Contributor Author

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.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@stale stale bot added the inactive label May 25, 2024
@frappe frappe deleted a comment from stale bot May 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-tests This PR needs automated unit-tests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

When creating a Purchase Order from a Sales Order, supplier's default payment terms template isn't being used
2 participants