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: delivery slip kit packaging different Uoms components #161920

Conversation

HANNICHE-Walid
Copy link
Member

Steps to reproduce:

  • Create a kit with components Uom set as Kg
  • Create an SO for the kit using packaging
  • Validate the delivery and try printing the delivery slip ERROR

Bug:
when generating the delivery slip we convert movelines qty to the packaging Uom but in the case of a kit the movelines are those of the components (in this we're converting Kg to units)

Fix:
Ignore kit lines (will be removed any way in _get_aggregated_product_quantities)

opw-3758203

@robodoo
Copy link
Contributor

robodoo commented Apr 15, 2024

Pull request status dashboard.

@C3POdoo C3POdoo added the OE the report is linked to a support ticket (opw-...) label Apr 15, 2024
@HANNICHE-Walid HANNICHE-Walid force-pushed the 17.0-opw-3758203-kitPdeliverySlip-waha branch 3 times, most recently from 80d5dca to 001252d Compare April 17, 2024 11:30
@HANNICHE-Walid HANNICHE-Walid marked this pull request as ready for review April 17, 2024 11:30
@C3POdoo C3POdoo requested a review from a team April 17, 2024 11:32
Steps to reproduce:
- Create a kit with components Uom set as Kg
- Create an SO for the kit using packaging
- Validate the delivery and try printing the delivery slip
ERROR

Bug:
when generating the delivery slip we convert movelines qty to the
packaging Uom but in the case of a kit the movelines are those of
the components (in this we're converting Kg to units)

Fix:
Ignore kit lines (will be removed any way in _get_aggregated_product_quantities)

opw-3758203
@HANNICHE-Walid HANNICHE-Walid force-pushed the 17.0-opw-3758203-kitPdeliverySlip-waha branch from 001252d to 47da1ec Compare May 23, 2024 10:00
@Whenrow
Copy link
Contributor

Whenrow commented May 24, 2024

robodoo r+

robodoo pushed a commit that referenced this pull request May 24, 2024
Steps to reproduce:
- Create a kit with components Uom set as Kg
- Create an SO for the kit using packaging
- Validate the delivery and try printing the delivery slip
ERROR

Bug:
when generating the delivery slip we convert movelines qty to the
packaging Uom but in the case of a kit the movelines are those of
the components (in this we're converting Kg to units)

Fix:
Ignore kit lines (will be removed any way in _get_aggregated_product_quantities)

opw-3758203

closes #161920

Signed-off-by: William Henrotin (whe) <whe@odoo.com>
robodoo pushed a commit that referenced this pull request May 24, 2024
Steps to reproduce:
- Create a kit with components Uom set as Kg
- Create an SO for the kit using packaging
- Validate the delivery and try printing the delivery slip
ERROR

Bug:
when generating the delivery slip we convert movelines qty to the
packaging Uom but in the case of a kit the movelines are those of
the components (in this we're converting Kg to units)

Fix:
Ignore kit lines (will be removed any way in _get_aggregated_product_quantities)

opw-3758203

closes #161920

Signed-off-by: William Henrotin (whe) <whe@odoo.com>
@robodoo robodoo closed this in ae70520 May 24, 2024
@fw-bot
Copy link
Contributor

fw-bot commented May 28, 2024

@HANNICHE-Walid @Whenrow this pull request has forward-port PRs awaiting action (not merged or closed):

@fw-bot
Copy link
Contributor

fw-bot commented May 29, 2024

@HANNICHE-Walid @Whenrow this pull request has forward-port PRs awaiting action (not merged or closed):

2 similar comments
@fw-bot
Copy link
Contributor

fw-bot commented May 30, 2024

@HANNICHE-Walid @Whenrow this pull request has forward-port PRs awaiting action (not merged or closed):

@fw-bot
Copy link
Contributor

fw-bot commented May 31, 2024

@HANNICHE-Walid @Whenrow this pull request has forward-port PRs awaiting action (not merged or closed):

@fw-bot
Copy link
Contributor

fw-bot commented Jun 1, 2024

@HANNICHE-Walid @Whenrow this pull request has forward-port PRs awaiting action (not merged or closed):

@fw-bot
Copy link
Contributor

fw-bot commented Jun 2, 2024

@HANNICHE-Walid @Whenrow this pull request has forward-port PRs awaiting action (not merged or closed):

2 similar comments
@fw-bot
Copy link
Contributor

fw-bot commented Jun 3, 2024

@HANNICHE-Walid @Whenrow this pull request has forward-port PRs awaiting action (not merged or closed):

@fw-bot
Copy link
Contributor

fw-bot commented Jun 5, 2024

@HANNICHE-Walid @Whenrow this pull request has forward-port PRs awaiting action (not merged or closed):

lohwswilson pushed a commit to lohwswilson/odoo that referenced this pull request Jun 5, 2024
Steps to reproduce:
- Create a kit with components Uom set as Kg
- Create an SO for the kit using packaging
- Validate the delivery and try printing the delivery slip
ERROR

Bug:
when generating the delivery slip we convert movelines qty to the
packaging Uom but in the case of a kit the movelines are those of
the components (in this we're converting Kg to units)

Fix:
Ignore kit lines (will be removed any way in _get_aggregated_product_quantities)

opw-3758203

closes odoo#161920

Signed-off-by: William Henrotin (whe) <whe@odoo.com>
@fw-bot fw-bot deleted the 17.0-opw-3758203-kitPdeliverySlip-waha branch June 7, 2024 14:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OE the report is linked to a support ticket (opw-...)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants