[18.0][MIG] product_variant_sale_price: Migration to 18.0 #374
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Hi @NICO-SOLUTIONS. May I ask if you are still working on this, or if ther are any updates? |
Feel free to review... |
|
/ocabot migration product_variant_sale_price |
0ea92e0 to
14d1f7f
Compare
|
This PR has the |
micheledic
approved these changes
May 12, 2025
AlienAtSystem
suggested changes
May 14, 2025
AlienAtSystem
approved these changes
May 14, 2025
…ate is created [product_variant_sale_price] Set minimun fix_price among variants to ensure consistency with the price shown in the shop
…ariants Steps to reproduce: - create more than one variant - change fix_price of one variant Current behavior: - lst_price of all variants are changed Expected behavior: - lst_price of only selected variant is changed
Currently translated at 100.0% (6 of 6 strings) Translation: product-variant-14.0/product-variant-14.0-product_variant_sale_price Translate-URL: https://translation.odoo-community.org/projects/product-variant-14-0/product-variant-14-0-product_variant_sale_price/pt/
Currently translated at 83.3% (5 of 6 strings) Translation: product-variant-15.0/product-variant-15.0-product_variant_sale_price Translate-URL: https://translation.odoo-community.org/projects/product-variant-15-0/product-variant-15-0-product_variant_sale_price/fr/
Currently translated at 100.0% (6 of 6 strings) Translation: product-variant-15.0/product-variant-15.0-product_variant_sale_price Translate-URL: https://translation.odoo-community.org/projects/product-variant-15-0/product-variant-15-0-product_variant_sale_price/ca/
Apply suggested changes from FrankC013
Currently translated at 100.0% (6 of 6 strings) Translation: product-variant-16.0/product-variant-16.0-product_variant_sale_price Translate-URL: https://translation.odoo-community.org/projects/product-variant-16-0/product-variant-16-0-product_variant_sale_price/es/
Currently translated at 100.0% (6 of 6 strings) Translation: product-variant-16.0/product-variant-16.0-product_variant_sale_price Translate-URL: https://translation.odoo-community.org/projects/product-variant-16-0/product-variant-16-0-product_variant_sale_price/it/
FIX [UPD] Update product_variant_sale_price.pot [BOT] post-merge updates Update translation files Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: product-variant-17.0/product-variant-17.0-product_variant_sale_price Translate-URL: https://translation.odoo-community.org/projects/product-variant-17-0/product-variant-17-0-product_variant_sale_price/ Translated using Weblate (Italian) Currently translated at 100.0% (7 of 7 strings) Translation: product-variant-17.0/product-variant-17.0-product_variant_sale_price Translate-URL: https://translation.odoo-community.org/projects/product-variant-17-0/product-variant-17-0-product_variant_sale_price/it/ Translated using Weblate (Spanish) Currently translated at 100.0% (7 of 7 strings) Translation: product-variant-17.0/product-variant-17.0-product_variant_sale_price Translate-URL: https://translation.odoo-community.org/projects/product-variant-17-0/product-variant-17-0-product_variant_sale_price/es/
[BOT] post-merge updates
…tion - _create_variant_ids(): Variants receive fix_price directly when created - write(): Efficiently updates fix_price of variants when `list_price` is changed - Removes unnecessary _update_fix_price() calls for better performance
In Odoo 18, the keyword argument pricelist was removed from the _get_combination_info method. This PR removes the parameter from the calls to avoid a TypeError
2a05e30 to
a8f3e41
Compare
- The singleton error in _create_variant_ids will be avoided by iterating over self
fda6755 to
0110a88
Compare
|
LGTM |
|
This PR has the |
| return res | ||
|
|
||
|
|
||
| class ProductProduct(models.Model): |
There was a problem hiding this comment.
@rousseldenis Hi Denis, i didn't found an answer in the contribution guide. But should the two classes be split into two files?
There was a problem hiding this comment.
just let me know if you want to have this separated
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
standard migration