[14.0] stock_average_daily_sale: returns handling #1
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
sebalix
force-pushed
the
14.0-stock_average_daily_sale-returns_handling
branch
3 times, most recently
from
8c90cfc to
b088b33
Compare
sebalix
force-pushed
the
14.0-stock_average_daily_sale-returns_handling
branch
from
b088b33 to
7bb974d
Compare
twalter-c2c
force-pushed
the
14.0-stock_average_daily_sale-returns_handling
branch
3 times, most recently
from
f3e6849 to
1c7220a
Compare
henrybackman
reviewed
Dec 9, 2024
sebalix
commented
Dec 9, 2024
There was a problem hiding this comment.
@twalter-c2c the new help should be done through a field override in customer specific modules (I guess they have been updated in that regards, not to be proposed to OCA?).
twalter-c2c
force-pushed
the
14.0-stock_average_daily_sale-returns_handling
branch
from
1c7220a to
03abbfa
Compare
henrybackman
requested changes
Dec 10, 2024
Take returns into account while calculating average daily sale usage. In our specific case, returns mean stock moves from the 'Inventory adjustment' location to the physical stock location (of usage 'internal'). Co-Authored-By: Sébastien Alix <[email protected]>
twalter-c2c
force-pushed
the
14.0-stock_average_daily_sale-returns_handling
branch
2 times, most recently
from
e57925c to
99e272d
Compare
twalter-c2c
force-pushed
the
14.0-stock_average_daily_sale-returns_handling
branch
from
99e272d to
86935c1
Compare
twalter-c2c
changed the base branch from
14.0-mig-stock_average_daily_sale
to
14.0
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Not ready to be proposed on OCA, so this PR was open to ease internal reviews.
Based on: