-
-
Notifications
You must be signed in to change notification settings - Fork 266
[MIG] stock_account_valuation_report: Migration to 18.0 #368
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
Conversation
067b468
to
54c5793
Compare
/ocabot migration stock_account_valuation_report |
class StockValuationHistory(models.TransientModel): | ||
_name = "stock.valuation.history" | ||
_description = "Stock Valuation History" | ||
|
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.
_rec_name = "inventory_datetime" | |
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.
I made same PR for 16.0 at #378
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.
PIcked from #378, thanks.
compute="_compute_inventory_value", | ||
search="_search_qty_discrepancy", | ||
) | ||
valuation = fields.Selection( |
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.
Reviewed and understood reason for deletion. Field is now related on product.template to categ_id and directly accessible to product.
Currently translated at 100.0% (13 of 13 strings) Translation: stock-logistics-reporting-12.0/stock-logistics-reporting-12.0-stock_account_valuation_report Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-reporting-12-0/stock-logistics-reporting-12-0-stock_account_valuation_report/es_MX/
[IMP] stock_account_valuation_report: filter by discrepancies
Currently translated at 100.0% (13 of 13 strings) Translation: stock-logistics-reporting-15.0/stock-logistics-reporting-15.0-stock_account_valuation_report Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-reporting-15-0/stock-logistics-reporting-15-0-stock_account_valuation_report/sl/
…e into account decimal precision
… filtering products with discrepancies also add a filter to see all discrepancies at once
…isplayed with AND conditions
Currently translated at 100.0% (37 of 37 strings) Translation: stock-logistics-reporting-16.0/stock-logistics-reporting-16.0-stock_account_valuation_report Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-reporting-16-0/stock-logistics-reporting-16-0-stock_account_valuation_report/it/
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.
I actually found these in the discrepancy module today, maybe we need it here too https://github.com/OCA/stock-logistics-reporting/pull/377/files#diff-d6055728635c3de730f384efe25ffbea9ac93cf0417f7337efb3b2a7fb88510e
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, static review
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.
Functional review and ok with the changes in code
This PR has the |
1 similar comment
This PR has the |
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
/ocabot merge nobump |
What a great day to merge this nice PR. Let's do it! |
Congratulations, your PR was merged at f538c0c. Thanks a lot for contributing to OCA. ❤️ |
Includes #378