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: getting scrap items from sub assemblies by fetching scrap items for parent BOM #4

Open
wants to merge 5 commits into
base: version-15-hotfix
Choose a base branch
from

Conversation

Carlos9k
Copy link

@Carlos9k Carlos9k commented Aug 19, 2024

Problem:

  • when creating a Stock Entry from a Work Order (click on finish button on Work Order and enter Qty for Manufacture and click Create). Specifically, when the "Set Operating Cost / Scrap Items From Sub-assemblies" option is enabled in Manufacturing Settings, the Stock Entry fails to correctly fetch the scrap items for BOM Item that on Work Order.

  • The problem occurs because the scrap items from sub-assemblies do not consider the BOM Scrap Item of the Parent BOM (fieldlabel : BOM No ) in the Work Order. This leads to incorrect stock entries.

How to reproduce the bug 🐛 :

scrap_item_issue_man_settings
issue_wo_details
issue_bom_scrap_items
issue_finish_button
issue_stock_entry_without_scraped_items

Proposed Solution

This update will ensure that the Stock Entry accurately considers the Parent BOM's scrap items when generating stock entries from Work Orders.

  • Stock Entry after fix ⬇️ :
    issue_after_fix

@Carlos9k Carlos9k requested a review from meaziz August 19, 2024 08:33
@KerollesFathy KerollesFathy changed the title 🛠️ fix: BOM scrap issue by fetching scrap items for parent BOM fix: getting scrap items from sub assemblies by fetching scrap items for parent BOM Aug 26, 2024
@KerollesFathy KerollesFathy force-pushed the Fix-BOM-Scrap branch 2 times, most recently from 144b5a5 to f2e1610 Compare December 11, 2024 12:10
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.

2 participants