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 - Add condition to delete product #27424

Closed

Conversation

atm-lucasmantegari
Copy link
Contributor

FIX - Add condition to delete product

We can currently delete a product even though it is in a BOM.
Adding this check helps avoid an SQL error when using this product on a OF.

@atm-lucasmantegari
Copy link
Contributor Author

@eldy I think Travis has a bug. If not, please explain to me

@eldy eldy added the PR to fix or conflict to solve PR needs to be fixed to be integrated (except for conflicts, a comment describes the fix to do) label Jan 12, 2024
eldy and others added 27 commits March 28, 2024 14:57
FIX : MRP divide product not working
BugFix Autocomplete under dialog box
Fix php error in cronjob.class.php
@atm-lucasmantegari
Copy link
Contributor Author

@eldy

@eldy
Copy link
Member

eldy commented Jul 1, 2024

@eldy

See my first comment..

@fappels
Copy link
Contributor

fappels commented Jul 2, 2024

See f401ad5
Is for MO, also BOM need to be added.

@atm-lucasmantegari
Copy link
Contributor Author

@eldy its better like this ?

@eldy
Copy link
Member

eldy commented Jul 17, 2024

@eldy its better like this ?

Seems better. But have cti errors

@eldy eldy added PR to fix or conflict to solve PR needs to be fixed to be integrated (except for conflicts, a comment describes the fix to do) and removed Discussion Some questions or discussions are opened and wait answers of author or other people to be processed labels Jul 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR to fix or conflict to solve PR needs to be fixed to be integrated (except for conflicts, a comment describes the fix to do)
Projects
None yet
Development

Successfully merging this pull request may close these issues.