-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
[16.0] [IMP] sale_mrp_bom: ensure bom_id can only be selected if right routing (manufacturing) is settle #3405
base: 16.0
Are you sure you want to change the base?
Conversation
fc733ba
to
b4015e9
Compare
b4015e9
to
8f5f57a
Compare
@arielbarreiros96 dear friend could you test this approach? I'm doing the bom_id field readonly if the routing is not properly settle. |
…ne if product's manufacture routes aren't properly settle
8f5f57a
to
4f89b73
Compare
As a matter of fact, I believe this proposal aligns with the add-on's original intent, as stated in the usage section of the README: 'if the routing is manufacturing then...' |
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.
From a functional POV, for me, it seems quite better, now it doesn't allow you to select BoM on the sale.order.line unless the product has the manufacture and MTO routes. I can't recall a case in which I needed to set the BoM without using MTO.
I wasn't able to test the behavior when stock_mts_mto_rule is installed, which is slightly different from pure MTO
thank U @arielbarreiros96 , do you mean this one? |
Fixes #3403.