-
-
Notifications
You must be signed in to change notification settings - Fork 402
[18.0] [MIG] stock_landed_costs_analytic: Migration to 18.0 #796
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
base: 18.0
Are you sure you want to change the base?
Conversation
Currently translated at 100.0% (4 of 4 strings) Translation: account-analytic-13.0/account-analytic-13.0-stock_landed_costs_analytic Translate-URL: https://translation.odoo-community.org/projects/account-analytic-13-0/account-analytic-13-0-stock_landed_costs_analytic/es/
Currently translated at 60.0% (3 of 5 strings) Translation: account-analytic-16.0/account-analytic-16.0-stock_landed_costs_analytic Translate-URL: https://translation.odoo-community.org/projects/account-analytic-16-0/account-analytic-16-0-stock_landed_costs_analytic/it/
Currently translated at 100.0% (5 of 5 strings) Translation: account-analytic-16.0/account-analytic-16.0-stock_landed_costs_analytic Translate-URL: https://translation.odoo-community.org/projects/account-analytic-16-0/account-analytic-16-0-stock_landed_costs_analytic/it/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: account-analytic-16.0/account-analytic-16.0-stock_landed_costs_analytic Translate-URL: https://translation.odoo-community.org/projects/account-analytic-16-0/account-analytic-16-0-stock_landed_costs_analytic/
Currently translated at 100.0% (6 of 6 strings) Translation: account-analytic-16.0/account-analytic-16.0-stock_landed_costs_analytic Translate-URL: https://translation.odoo-community.org/projects/account-analytic-16-0/account-analytic-16-0-stock_landed_costs_analytic/it/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: account-analytic-16.0/account-analytic-16.0-stock_landed_costs_analytic Translate-URL: https://translation.odoo-community.org/projects/account-analytic-16-0/account-analytic-16-0-stock_landed_costs_analytic/
Currently translated at 100.0% (6 of 6 strings) Translation: account-analytic-16.0/account-analytic-16.0-stock_landed_costs_analytic Translate-URL: https://translation.odoo-community.org/projects/account-analytic-16-0/account-analytic-16-0-stock_landed_costs_analytic/it/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: account-analytic-16.0/account-analytic-16.0-stock_landed_costs_analytic Translate-URL: https://translation.odoo-community.org/projects/account-analytic-16-0/account-analytic-16-0-stock_landed_costs_analytic/
Currently translated at 100.0% (8 of 8 strings) Translation: account-analytic-16.0/account-analytic-16.0-stock_landed_costs_analytic Translate-URL: https://translation.odoo-community.org/projects/account-analytic-16-0/account-analytic-16-0-stock_landed_costs_analytic/it/
Currently translated at 100.0% (8 of 8 strings) Translation: account-analytic-17.0/account-analytic-17.0-stock_landed_costs_analytic Translate-URL: https://translation.odoo-community.org/projects/account-analytic-17-0/account-analytic-17-0-stock_landed_costs_analytic/es/
self.product = self.Product.create( | ||
{ | ||
"name": "Product Test", | ||
"type": "product", |
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.
"type": "product", | |
"type": "consu", | |
"is_storable": True, |
Please mate, add this to fix tests.
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.
@luisDIXMIT Thanks Luis, Totally right. I already made it!
The PR title name must be: [18.0] [MIG] stock_landed_costs_analytic: Migration to 18.0 . It's a convenion. |
6396547
to
15fbd29
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.
Tested on runboat and LGTM. Great job 👍
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.
Code review LG!
No description provided.