Skip to content

[19.0][FIX] stock_product_demand_info: exclude today from demand periods#504

Merged
OCA-git-bot merged 1 commit into
OCA:19.0from
camptocamp:fix-stock_product_demand_info
Jun 9, 2026
Merged

[19.0][FIX] stock_product_demand_info: exclude today from demand periods#504
OCA-git-bot merged 1 commit into
OCA:19.0from
camptocamp:fix-stock_product_demand_info

Conversation

@yankinmax

Copy link
Copy Markdown

Fixes the replenishment “Demand by period” so historical demand excludes stock moves of today.

Real issue case: when checking a product in the replenishment view, validating an outgoing move today immediately increased the “Year to date” demand value.
This made YTD look like completed historical demand had changed during the day, while today’s demand is still in progress.
The demand period domain now caps history at date < today.

Hello @ivantodorovich , can you pls take a look?

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hi @ivantodorovich,
some modules you are maintaining are being modified, check this out!

@OCA-git-bot OCA-git-bot added series:19.0 mod:stock_product_demand_info Module stock_product_demand_info labels Jun 9, 2026

@vvrossem vvrossem left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ivantodorovich ivantodorovich left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/ocabot merge minor

@OCA-git-bot

Copy link
Copy Markdown
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 19.0-ocabot-merge-pr-504-by-ivantodorovich-bump-minor, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 4831d88 into OCA:19.0 Jun 9, 2026
7 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at f8e15c9. Thanks a lot for contributing to OCA. ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants