-
-
Notifications
You must be signed in to change notification settings - Fork 239
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][FIX] stock_account_valuation_report: improve searching on discrepancies #338
[16.0][FIX] stock_account_valuation_report: improve searching on discrepancies #338
Conversation
…e into account decimal precision
e356599
to
3e0bcbb
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
apart from comment made by Miquel, LGTM
f717767
to
939f7f0
Compare
… filtering products with discrepancies also add a filter to see all discrepancies at once
939f7f0
to
d0c56d2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/ocabot merge patch
Hey, thanks for contributing! Proceeding to merge this for you. |
Congratulations, your PR was merged at 4cd924f. Thanks a lot for contributing to OCA. ❤️ |
Otherwise we can get False positive results when applying the filters:
cc@ForgeFlow