-
-
Notifications
You must be signed in to change notification settings - Fork 507
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][FIX] mrp_multi_level: fix kit/phantom planning #1367
[16.0][FIX] mrp_multi_level: fix kit/phantom planning #1367
Conversation
Hi @LoisRForgeFlow, @ChrisOForgeFlow, @JordiBForgeFlow, |
266c958
to
be70efc
Compare
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.
@matt454357 Good catch. Please add ForgeFlow@5ae917c
fixes OCA#1362 Ignoring qty_available for phantom products prevents double counting the qty_available of components. Creating planned orders for phantom products is simpler than recursively exploding phantom BOMs. This also makes it easier to analyze the planning data generated by the MRP calculation.
be70efc
to
33cf4af
Compare
@JordiBForgeFlow |
@LoisRForgeFlow What do you think about this? |
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.
Both changes make sense. Tested 👍
/ocabot merge patch |
What a great day to merge this nice PR. Let's do it! |
Congratulations, your PR was merged at 70ff84f. Thanks a lot for contributing to OCA. ❤️ |
fixes #1362
Ignoring qty_available for phantom products prevents double counting the qty_available of components.
Creating planned orders for phantom products is simpler than recursively exploding phantom BOMs. This also makes it easier to analyze the planning data generated by the MRP calculation.