Skip to content

fix: better validation message with solution for BOM recursion (backport #47472) #47477

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

Merged

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented May 8, 2025

If the user wants to use a finished good as a raw material, and another BOM already exists for it, the system does not allow creating a new BOM. In this case, the user should enable the 'Do Not Explode' checkbox in the line item, which mostly users missed. Due to which they think that the finished good can't be use a as a Raw Material. So to address this, the validation message has added the solution.

Screenshot 2025-05-08 at 5 43 00 PM
This is an automatic backport of pull request #47472 done by [Mergify](https://mergify.com).

@mergify mergify bot requested a review from rohitwaghchaure as a code owner May 8, 2025 15:10
@github-actions github-actions bot added the needs-tests This PR needs automated unit-tests. label May 8, 2025
@rohitwaghchaure rohitwaghchaure merged commit a450f4c into version-15-hotfix May 20, 2025
12 of 13 checks passed
@rohitwaghchaure rohitwaghchaure deleted the mergify/bp/version-15-hotfix/pr-47472 branch May 20, 2025 10:45
frappe-pr-bot pushed a commit that referenced this pull request May 20, 2025
# [15.62.0](v15.61.1...v15.62.0) (2025-05-20)

### Bug Fixes

* alias name and parent to prevent child row mapping issues ([612fa7c](612fa7c))
* allow FG as RM by default (backport [#47543](#47543)) ([#47550](#47550)) ([9355782](9355782))
* asset cancellation issue (backport [#47639](#47639)) ([#47641](#47641)) ([ce9da48](ce9da48))
* asset image field updation issue (backport [#47615](#47615)) ([#47617](#47617)) ([35c7af1](35c7af1))
* better validation message with solution for BOM recursion (backport [#47472](#47472)) ([#47477](#47477)) ([a450f4c](a450f4c))
* Broken test + use `super()` appropriately ([5b50d5a](5b50d5a))
* Conflicts ([9cede83](9cede83))
* correct expense amount in party ledger summary. ([09a46fc](09a46fc))
* date formatting in process_statement_of_accounts accounts_receivable print format ([cf354c0](cf354c0))
* include only invoices with update_stock = 0  for billed amt in delivery note. ([70e190d](70e190d))
* incorrect qty during reset (backport [#47593](#47593)) ([#47595](#47595)) ([72ae80e](72ae80e))
* mapping of dispatch address when creating PO from SO (backport [#47552](#47552)) ([#47553](#47553)) ([30ec69c](30ec69c))
* POS Invoice can't use Loyalty Points when Global Rounded Total is Disabled (backport [#47491](#47491)) ([#47564](#47564)) ([926c0c5](926c0c5))
* pos item group filter fetching wrong items (backport [#47545](#47545)) ([#47546](#47546)) ([5a3eff0](5a3eff0))
* **quotation:** use `Text Editor` field in alternative items dialog for item description ([32eeeda](32eeeda))
* remove hardcoded doctype in `make_return_doc` ([1a69d81](1a69d81))
* removed invalid child param to prevent callback failure ([073d06c](073d06c))
* **SalesAnalytics:** Ignore opening entries ([be280a4](be280a4))
* set no_copy to party_balance field in Payment Entry ([da4ed5c](da4ed5c))
* set no_copy to party_balance field in Payment Entry ([52cab02](52cab02))
* validate inter-company transaction address links ([86aa072](86aa072))
* validation message format (backport [#47542](#47542)) ([#47549](#47549)) ([f225e19](f225e19))

### Features

* add checbox for validating time logs in job card ([80c7661](80c7661))
* add option to calculate ageing based on report date or today date ([69337cf](69337cf))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-tests This PR needs automated unit-tests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant