[19.0][MIG] product_pricelist_revision: Migration to 19.0#2100
Conversation
|
/ocabot migration product_pricelist_revision |
gdgellatly
left a comment
There was a problem hiding this comment.
Functional test on own db plus code review LGTM.
gdgellatly
left a comment
There was a problem hiding this comment.
Sorry I put review against wrong module. I now tested this one on live database and it didn't work. It duplicated the item just fine but did not take the variance. Will get back to tomorrow.
[UPD] Update product_pricelist_revision.pot [UPD] README.rst Update translation files Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: product-attribute-12.0/product-attribute-12.0-product_pricelist_revision Translate-URL: https://translation.odoo-community.org/projects/product-attribute-12-0/product-attribute-12-0-product_pricelist_revision/
…t filter TT28144 [UPD] Update product_pricelist_revision.pot
Currently translated at 100.0% (30 of 30 strings) Translation: product-attribute-13.0/product-attribute-13.0-product_pricelist_revision Translate-URL: https://translation.odoo-community.org/projects/product-attribute-13-0/product-attribute-13-0-product_pricelist_revision/ca/
Before this change on runbot we caught a Warning about deprecated call to decimal_precision.get_precision(<application>) [UPD] README.rst product_pricelist_revision 13.0.1.1.0
Fix website key in the manifests
Currently translated at 31.0% (9 of 29 strings) Translation: product-attribute-14.0/product-attribute-14.0-product_pricelist_revision Translate-URL: https://translation.odoo-community.org/projects/product-attribute-14-0/product-attribute-14-0-product_pricelist_revision/it/
Currently translated at 100.0% (29 of 29 strings) Translation: product-attribute-16.0/product-attribute-16.0-product_pricelist_revision Translate-URL: https://translation.odoo-community.org/projects/product-attribute-16-0/product-attribute-16-0-product_pricelist_revision/it/
- Include context keys for avoiding mail operations overhead.
Currently translated at 100.0% (29 of 29 strings) Translation: product-attribute-16.0/product-attribute-16.0-product_pricelist_revision Translate-URL: https://translation.odoo-community.org/projects/product-attribute-16-0/product-attribute-16-0-product_pricelist_revision/fr/
1b96722 to
68cfb1b
Compare
Hi @gdgellatly, I have tested this module locally and on Runbot, and it is working as expected. Please note that this module's duplicate wizard only works with fixed price items (compute_price = 'fixed'). It does not apply the variance for other pricing types such as formula or percentage/discount. The wizard view also mentions this: "Note: Only works with fixed price items. The variation % will be applied to calculate the new price." If you are facing any issue specifically with fixed price items, could you please elaborate on the steps to reproduce and where exactly you are getting stuck? That would help us investigate further. |
Hi @BhaveshHeliconia, yes sorry we worked that out and deleted the module. We have no use case for it so will remove my review. EDIT: I no longr have access to remove reviews so we will just have to live with it, it did work, just not how expected. |
CarlosRoca13
left a comment
There was a problem hiding this comment.
Please add this change: #2291
…test Odoo 18 enforces a strict date_end > date_start constraint on pricelist items. The test was using the same datetime for both fields, causing a ValidationError. See https://github.com/odoo/odoo/blob/36a7cadbfb9007ebddd660b7709e4bef58e96c2d/addons/product/models/product_pricelist_item.py#L307
68cfb1b to
76d44c4
Compare
Done, thanks! |
|
/ocabot merge nobump |
|
What a great day to merge this nice PR. Let's do it! |
|
Congratulations, your PR was merged at 3ae2dd5. Thanks a lot for contributing to OCA. ❤️ |
No description provided.