Skip to content
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

[17.0][MIG] sale_variant_configurator: Migration to 17.0 #381

Open
wants to merge 25 commits into
base: 17.0
Choose a base branch
from

Conversation

HeliconiaSolutions
Copy link

No description provided.

pedrobaeza and others added 25 commits January 13, 2025 17:40
This comes from module sale_product_variants from odoomrp/odoomrp-wip.
- saleable products for Sale Orders.
- purchable products for Purchase Orders.
Depending on the onchange order loaded in runtime, description put in the SO line
might be the one got from the regular way. With this, we assure we get the proper
description, and also in the partner language (using the trick with the partner field).
* Use generic method for creating the variant if needed
* Create the variant when the sales is already confirmed
for some reason, after the creation of the product, the sale.order.line
was emptied (e.g. product_tmpl_id was False) -> the onchange would do nothing.

I split the test in two, and in the second test, I create the product before
creating the sale order line.
…omer language

As the model `sale.order.line` doesn't have `partner_id` field, the
description is not populated in the customer language, as the automatism
put in product_configurator makes use of this trick.

For not adding such field that can conflict with other parts, let's
overwrite the methods that populate the description and pass upstream
the customer language in the context.

TT33029
Currently translated at 100.0% (8 of 8 strings)

Translation: product-variant-16.0/product-variant-16.0-sale_variant_configurator
Translate-URL: https://translation.odoo-community.org/projects/product-variant-16-0/product-variant-16-0-sale_variant_configurator/pt_BR/
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.