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: partial return on POS invoice #43253

Merged
merged 2 commits into from
Sep 23, 2024

Conversation

Nihantra-Patel
Copy link
Contributor

@Nihantra-Patel Nihantra-Patel commented Sep 17, 2024

version 15 and 14

fixes: #37998 & https://discuss.frappe.io/t/cant-submit-a-partial-return-on-pos-invoice/133331?u=ncp

Before:

before_pos_return.mp4

After:

after_pos_return.mp4

@github-actions github-actions bot added the needs-tests This PR needs automated unit-tests. label Sep 17, 2024
@Nihantra-Patel Nihantra-Patel removed the needs-tests This PR needs automated unit-tests. label Sep 17, 2024
@Mina-Geez
Copy link

how can i pull it into my windows docker container??

@Nihantra-Patel
Copy link
Contributor Author

no idea in windows docker container side, you can check the code What have I changed.

@ruthra-kumar ruthra-kumar self-assigned this Sep 23, 2024
@ruthra-kumar ruthra-kumar merged commit 3e13a43 into frappe:develop Sep 23, 2024
14 checks passed
Nihantra-Patel added a commit that referenced this pull request Sep 23, 2024
…-43253

fix: partial return on POS invoice (backport #43253)
Nihantra-Patel added a commit that referenced this pull request Sep 23, 2024
…-43253

fix: partial return on POS invoice (backport #43253)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

POS Return error "Total payments amount can't be greater than xx.x"
3 participants