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] mrp: Printing a BOM with a variant #32128

Closed
wants to merge 1 commit into from

Conversation

Projects
None yet
3 participants
@simongoffin
Copy link
Contributor

simongoffin commented Mar 26, 2019

Steps to reproduce the bug:

  • Create a BOM for product P
  • Generate variants for P: P1 and P2
  • Go to BOM of P and click on Structure & Cost
  • Change the current variant to P2
  • Click on "Print"

Bug:

A traceback was raised because the function _get_bom expects in arguments that
product_id is an id and not a recordset.

opw:1950401

[FIX] mrp: Printing a BOM with a variant
Steps to reproduce the bug:
- Create a BOM for product P
- Generate variants for P: P1 and P2
- Go to BOM of P and click on Structure & Cost
- Change the current variant to P2
- Click on "Print"

Bug:

A traceback was raised because the function _get_bom expects in arguments that
product_id is an id and not a recordset.

opw:1950401

@robodoo robodoo added the seen 🙂 label Mar 26, 2019

@C3POdoo C3POdoo added the OE label Mar 26, 2019

@robodoo robodoo added the CI 🤖 label Mar 26, 2019

@simongoffin

This comment has been minimized.

Copy link
Contributor Author

simongoffin commented Mar 28, 2019

robodoo r+

@robodoo

This comment has been minimized.

Copy link
Contributor

robodoo commented Mar 28, 2019

Merged, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.