Skip to content

[19.0][MIG] product_warranty: Migration to 19.0#506

Open
fredericgrall wants to merge 67 commits intoOCA:19.0from
apikcloud:19.0-mig-product_warranty
Open

[19.0][MIG] product_warranty: Migration to 19.0#506
fredericgrall wants to merge 67 commits intoOCA:19.0from
apikcloud:19.0-mig-product_warranty

Conversation

@fredericgrall
Copy link
Copy Markdown

No description provided.

manu and others added 30 commits November 17, 2025 09:50
…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)
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/
weblate and others added 6 commits November 17, 2025 09:50
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/
@fredericgrall fredericgrall mentioned this pull request Nov 17, 2025
3 tasks
@fredericgrall fredericgrall changed the title [19.0][MIG] product_warranty: Migretion to 19.0 [19.0][MIG] product_warranty: Migration to 19.0 Nov 17, 2025
@fredericgrall fredericgrall force-pushed the 19.0-mig-product_warranty branch 2 times, most recently from 81b5e7b to 7240627 Compare November 17, 2025 14:42
Copy link
Copy Markdown

@CasVissers-360ERP CasVissers-360ERP left a comment

Choose a reason for hiding this comment

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

Functional review

@rousseldenis
Copy link
Copy Markdown

/ocabot migration product_warranty

"return.instruction",
"Instructions",
default=_get_default_instructions,
default=lambda self: self.env["return.instruction"].search(
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@fredericgrall This change is maybe a little too much. I would have let the function but use the lamda (to avoid static declaration).

@DavidJForgeFlow
Copy link
Copy Markdown

@fredericgrall could you attend @rousseldenis feedback? Thanks!

@fredericgrall fredericgrall force-pushed the 19.0-mig-product_warranty branch from 7240627 to cd5e4a1 Compare April 29, 2026 13:29
@fredericgrall
Copy link
Copy Markdown
Author

@rousseldenis @DavidJForgeFlow , sorry for the delay. I 'm modified

Copy link
Copy Markdown

@DavidJForgeFlow DavidJForgeFlow left a comment

Choose a reason for hiding this comment

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

Minor comment, but the migration is fine!

):
partner_id = record.warranty_return_other_address.id
record.warranty_return_address = partner_id

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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.

@OCA-git-bot
Copy link
Copy Markdown
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.