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: Validate if item exists on uploading items in stock reco #27538
Conversation
- Uploading non existent item in stock reco and then changing warehouse or batch gave an error - Check for non existent item
This comment has been minimized.
This comment has been minimized.
erpnext/stock/doctype/stock_reconciliation/stock_reconciliation.py
Outdated
Show resolved
Hide resolved
erpnext/stock/doctype/stock_reconciliation/stock_reconciliation.py
Outdated
Show resolved
Hide resolved
erpnext/stock/doctype/stock_reconciliation/stock_reconciliation.py
Outdated
Show resolved
Hide resolved
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.
LGTM.
We can think of adding link field validations during/post CSV imports?
…#27538) * fix: Validate if item exists on uploading items in stock reco - Uploading non existent item in stock reco and then changing warehouse or batch gave an error - Check for non existent item * chore: translation Co-authored-by: Ankush Menat <ankushmenat@gmail.com> (cherry picked from commit 9b38888)
@ankush yes that will be good and also maybe a hook that can enable us to write custom checks too ? Because unlike data import this import doesn't give that error traceback bit (if i'm not wrong),and causes a lot of type issues too (historically) like comparison between |
@Mergifyio backport version-12-hotfix |
* fix: Validate if item exists on uploading items in stock reco - Uploading non existent item in stock reco and then changing warehouse or batch gave an error - Check for non existent item * chore: translation Co-authored-by: Ankush Menat <ankushmenat@gmail.com> (cherry picked from commit 9b38888)
Command
|
#27593) * fix: Validate if item exists on uploading items in stock reco - Uploading non existent item in stock reco and then changing warehouse or batch gave an error - Check for non existent item * chore: translation Co-authored-by: Marica <maricadsouza221197@gmail.com>
…#27538) * fix: Validate if item exists on uploading items in stock reco - Uploading non existent item in stock reco and then changing warehouse or batch gave an error - Check for non existent item * chore: translation Co-authored-by: Ankush Menat <ankushmenat@gmail.com>
Issue:
Uploading (via Upload button) non existent item in stock reco and then changing warehouse or batch (trigger JS action) gave an error: