agritheory / beam

General Purpose 2D barcode scanning for ERPNext
https://agritheory.com/documentation/beam/
Other
20 stars 9 forks source link

Unable to cancel SE for Manufacture when handling unit fields for all rows are left blank #86

Open HKuz opened 4 months ago

HKuz commented 4 months ago

Original issue (posted in Inventory Tools) described the problem as follows:

I've setup a FG part # with two outside vendor operations. Attached are the BOMs setup and details. The issue is material consumption for manufacture did not require handling units entry (the material transfer for manufacturing did). I tried to cancel the material consumption for manufacture entry and I have an error that is reflected on the attached sheet.

Subcontract BOM HU Transfer Issue.docx

To replicate this issue:

Some initial thoughts on solutions - open to discussion:

agritheory commented 4 months ago

bypass the recombine dialog if there are no HUs in the items

This should be the "bug fix", in combination with #43

on validate, check for a SE Transfer for Manufacture (for same WO) and if there are HUs on items. If present, warn / throw error to require that field for the current SE for Manufacture

This seems like an approach worth exploring, perhaps in combination with #43, which would give visibility for when an item should be using Handling Units.

take one step further, and try to fill in the HU - this could get messy if the order is completed with multiple stock entries for Manufacture, or if there are multiple item rows for the same raw material (if the user needed several HUs of material for the job)

I agree that this approach is a little fraught, knowing which HUs should be there could_ be determined with FIFO, but it's just a suggestion. A better approach for when you need to cancel something like this is to scan in the HUs.