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

[18.0][MIG] website_sale_product_pack: Migration to 18.0 #194

Open
wants to merge 31 commits into
base: 18.0
Choose a base branch
from

Conversation

ALopez-Adhoc
Copy link

No description provided.

@ALopez-Adhoc ALopez-Adhoc force-pushed the 18.0-mig-website_sale_product_pack branch 4 times, most recently from f8dd1ef to cc7e0f7 Compare December 26, 2024 19:41
@matiasperalta1 matiasperalta1 force-pushed the 18.0-mig-website_sale_product_pack branch from cc7e0f7 to b070ee4 Compare January 17, 2025 18:30
Copy link

@matiasperalta1 matiasperalta1 left a comment

Choose a reason for hiding this comment

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

LGTM

website_sale_product_pack/__init__.py Outdated Show resolved Hide resolved
website_sale_product_pack/controllers/__init__.py Outdated Show resolved Hide resolved
website_sale_product_pack/controllers/main.py Outdated Show resolved Hide resolved
website_sale_product_pack/controllers/variant.py Outdated Show resolved Hide resolved
website_sale_product_pack/models/product_template.py Outdated Show resolved Hide resolved
website_sale_product_pack/models/product_template.py Outdated Show resolved Hide resolved
@augusto-weiss
Copy link

Hi @ALopez-Adhoc i added some review... also edit the commit name please to [18.0][MIG] thanks a lot

@ALopez-Adhoc ALopez-Adhoc force-pushed the 18.0-mig-website_sale_product_pack branch from b070ee4 to 59167c4 Compare January 20, 2025 18:30
@ALopez-Adhoc ALopez-Adhoc changed the title [18.0][MIG} website_sale_product_pack: Migration to 18.0 [18.0][MIG] website_sale_product_pack: Migration to 18.0 Jan 20, 2025
@ALopez-Adhoc
Copy link
Author

@augusto-weiss Thanks for the review. The changes has been applied. You can review them again if you will

@ALopez-Adhoc ALopez-Adhoc force-pushed the 18.0-mig-website_sale_product_pack branch 4 times, most recently from fc791c8 to d95f90a Compare January 22, 2025 12:03
chienandalu and others added 13 commits January 23, 2025 10:10
Compatibility module between sale_product_pack and website_sale

TT30385
Done for e-commerce compatibility purposes, althoug it's more performant
indeed.
Create and select a specific pricelist for avoiding problems in
integrated environments where the default pricelist currency has been
changed.
When the cart is confirmed, a price recalculation is triggered for every
order line. This is wrong for detailed totalized packs, which lines
should be at 0.

TT38186
alvaro-domatix and others added 17 commits January 23, 2025 10:10
Currently translated at 100.0% (9 of 9 strings)

Translation: product-pack-16.0/product-pack-16.0-website_sale_product_pack
Translate-URL: https://translation.odoo-community.org/projects/product-pack-16-0/product-pack-16-0-website_sale_product_pack/it/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: product-pack-17.0/product-pack-17.0-website_sale_product_pack
Translate-URL: https://translation.odoo-community.org/projects/product-pack-17-0/product-pack-17-0-website_sale_product_pack/
Currently translated at 100.0% (9 of 9 strings)

Translation: product-pack-17.0/product-pack-17.0-website_sale_product_pack
Translate-URL: https://translation.odoo-community.org/projects/product-pack-17-0/product-pack-17-0-website_sale_product_pack/it/
@ALopez-Adhoc ALopez-Adhoc force-pushed the 18.0-mig-website_sale_product_pack branch from d95f90a to 1db6641 Compare January 23, 2025 13:19
website_sale_product_pack/models/website.py Outdated Show resolved Hide resolved
website_sale_product_pack/models/__init__.py Outdated Show resolved Hide resolved
@ALopez-Adhoc ALopez-Adhoc force-pushed the 18.0-mig-website_sale_product_pack branch from 1db6641 to ba06b65 Compare January 23, 2025 19:28
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.