Skip to content

[18.0][IMP] ddmrp: consider that done quantity and demand can be different in stock.move #528

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

Open
wants to merge 1 commit into
base: 18.0
Choose a base branch
from

Conversation

LoisRForgeFlow
Copy link
Contributor

@LoisRForgeFlow LoisRForgeFlow commented Mar 31, 2025

Since v17, quantity field represents the actual done quantity and
can differ from product_qty which remeains as the original demand
when validating a transfer without a backorder. This need to be considered by ddmrp.

To do:

  • : Create a generic module that computes stock.move's quantity in the uom of the product.
  • : Use it here to fix the provided test case.

@LoisRForgeFlow LoisRForgeFlow force-pushed the 18.0-imp-ddmrp-stock-move-quantity branch from d63ef9b to c1114fc Compare March 31, 2025 14:54
@LoisRForgeFlow LoisRForgeFlow changed the title [18.0][IMP] ddmrp: stock move quantity [18.0][IMP] ddmrp: consider that done quantity and demand can be different in stock.move Mar 31, 2025
…in stock.move

Since v17, `quantity` field represents the actual done quantity and
can differ from `product_qty` which remeains as the original demand
when validating a transfer without a backorder. This need to be
considered by ddmrp.
@ThiagoMForgeFlow ThiagoMForgeFlow force-pushed the 18.0-imp-ddmrp-stock-move-quantity branch from c1114fc to a06d7aa Compare April 15, 2025 10:05
@ThiagoMForgeFlow
Copy link
Contributor

Generic module that computes stock.move's quantity in the uom of the product:

@LoisRForgeFlow LoisRForgeFlow marked this pull request as ready for review April 23, 2025 13:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants