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

Auto close PO when SCO is Closed #38658

Closed
precihole opened this issue Dec 10, 2023 · 3 comments · Fixed by #38667
Closed

Auto close PO when SCO is Closed #38658

precihole opened this issue Dec 10, 2023 · 3 comments · Fixed by #38667
Assignees
Labels

Comments

@precihole
Copy link

Information about bug

Hello @rohitwaghchaure ,

When I return from the supplier, the subcontracting order is closed, but the purchase order remains unchanged.

Module

buying

Version

Frappe Framework: v14.58.0 (HEAD)

Installation method

None

Relevant log output / Stack trace / Full Error Message.

No response

@ophl55
Copy link

ophl55 commented Dec 12, 2023

I dont't think the PO should be closed automatically in that case. Also the SCO should not be closed automatically.
The docs should be set "Completed" automatically if everything went fine and all the required Receipts have been submitted.

The user should decide to close a pending business process and the user should do it manually.
There might be a lot of reason why you don't want the SCO to be closed automatically on SCO Return.

One example:
Maybe you need the material yourself or for another supplier and you will send the subcontractor new material as soon as possible so that he can fulfill his PO/SCO. In that case it would be wrong to close the SCO automatically by returning the material.

What do you think? @s-aga-r @precihole

@ophl55
Copy link

ophl55 commented Dec 12, 2023

Please also check here: #38702

@ophl55
Copy link

ophl55 commented Dec 12, 2023

Also automatically closing a preceding document is nowhere else done in ERPNext.
So to me this implementation appears heavily counter intuitive, it doesn't match with the rest of the design/ux.

It might be alright if the SCO gets closed when the user closes the PO because the SCO depends much more on the PO than the other way around.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 28, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants