[19.0][MIG] product_warranty: Migration to 19.0#506
[19.0][MIG] product_warranty: Migration to 19.0#506fredericgrall wants to merge 67 commits intoOCA:19.0from
Conversation
…s it seems to be lost in another module that is not here
…ier, in such case, returns the product to the company's address
- Add dependency sale because group "group_sale_salesman" and "group_sale_manager" are defined on modul sales_team * sale depend to sales_team * update ir.model.access base. -> sales_team. - Fix product.prod_config_main not compatible and moved to sale.prod_config_main - Make removed col="6 when add product supplier info field too small (width)
Duplicated msgid
Currently translated at 100.0% (41 of 41 strings) Translation: rma-12.0/rma-12.0-product_warranty Translate-URL: https://translation.odoo-community.org/projects/rma-12-0/rma-12-0-product_warranty/pt_BR/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: rma-17.0/rma-17.0-product_warranty Translate-URL: https://translation.odoo-community.org/projects/rma-17-0/rma-17-0-product_warranty/
Currently translated at 100.0% (40 of 40 strings) Translation: rma-18.0/rma-18.0-product_warranty Translate-URL: https://translation.odoo-community.org/projects/rma-18-0/rma-18-0-product_warranty/it/
Currently translated at 100.0% (40 of 40 strings) Translation: rma-18.0/rma-18.0-product_warranty Translate-URL: https://translation.odoo-community.org/projects/rma-18-0/rma-18-0-product_warranty/it/
81b5e7b to
7240627
Compare
|
/ocabot migration product_warranty |
| "return.instruction", | ||
| "Instructions", | ||
| default=_get_default_instructions, | ||
| default=lambda self: self.env["return.instruction"].search( |
There was a problem hiding this comment.
@fredericgrall This change is maybe a little too much. I would have let the function but use the lamda (to avoid static declaration).
|
@fredericgrall could you attend @rousseldenis feedback? Thanks! |
7240627 to
cd5e4a1
Compare
|
@rousseldenis @DavidJForgeFlow , sorry for the delay. I 'm modified |
DavidJForgeFlow
left a comment
There was a problem hiding this comment.
Minor comment, but the migration is fine!
| ): | ||
| partner_id = record.warranty_return_other_address.id | ||
| record.warranty_return_address = partner_id | ||
|
|
There was a problem hiding this comment.
It's a minor thing here, but you changed the location of the method and it's taken as changes for git. Maybe you could return it to it's previous place.
|
This PR has the |
No description provided.