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

[16.0][ADD] stock_move_value #1777

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

Conversation

AungKoKoLin1997
Copy link
Contributor

This module adds value fields to the stock move model, to add visibility of how the move has affected the stock valuation.

@qrtl QT4685

Copy link
Member

@yostashiro yostashiro left a comment

Choose a reason for hiding this comment

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

@AungKoKoLin1997 Can you please add a CONTEXT.md as the intention is probably not clear to most people.

"""
Odoo calculates the cost of outgoing stock moves based on the product's costing method, no matter what the cause of the operation is. This can be problematic especially when a user just reverts the purchase receipt operation which was done by mistake (e.g., processing the wrong receipt picking) for FIFO and AVCO products, as the returned value is not the same as the value of the original receipt. Some auditors might see such inconsistencies as the result of intended manipulation on the financial performance.

For FIFO products, you can use the OCA module stock_account_fifo_return_origin (yet to be added with #1649) to return the same value as the receipt. However, there is no complete solution for AVCO products.

This module aims to provide users with a means to identify discrepancies resulting from their purchase return operations, allowing them to take necessary actions (e.g., inventory revaluation) as appropriate.
"""

@AungKoKoLin1997 AungKoKoLin1997 force-pushed the 16.0-add-stock_move_value branch from c813214 to d601bc0 Compare November 25, 2024 09:21
@AungKoKoLin1997
Copy link
Contributor Author

@yostashiro I added the CONTEXT.md.

Copy link
Member

@yostashiro yostashiro left a comment

Choose a reason for hiding this comment

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

Code review and functional test.

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