Skip to content

[DO NOT MERGE] red-proof: future-ADU move-UoM aggregation bug (test only)#5

Closed
dnplkndll wants to merge 1 commit into
19.0from
19.0-redproof-adu-future-uom
Closed

[DO NOT MERGE] red-proof: future-ADU move-UoM aggregation bug (test only)#5
dnplkndll wants to merge 1 commit into
19.0from
19.0-redproof-adu-future-uom

Conversation

@dnplkndll

Copy link
Copy Markdown

Evidence branch: contains ONLY a failing test against the unmodified 19.0 ddmrp base — no production code changes. CI red on test_51_adu_calculation_future_actual_move_uom demonstrates the bug; the fix branch carries the same test green. Will be closed once the upstream PR cites this run.

…ation

Test only, no fix: demonstrates that _calc_adu_future_demand aggregates
product_uom_qty (move UoM) instead of product_qty (product UoM) on the
current 19.0 branch. A future delivery of 5 Dozen contributes 5 instead
of 60 to the ADU.
@dnplkndll dnplkndll force-pushed the 19.0-redproof-adu-future-uom branch from 076a940 to a5495a9 Compare June 11, 2026 11:38
@dnplkndll

Copy link
Copy Markdown
Author

Closing: evidence captured — upstream PR filed as OCA#612 citing the red run from this branch. Branch kept.

@dnplkndll dnplkndll closed this Jun 12, 2026
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.

1 participant