Skip to content

[19.0][MIG] product_pricelist_revision: Migration to 19.0#2100

Merged
OCA-git-bot merged 29 commits into
OCA:19.0from
HeliconiaIO:19.0-mig-product_pricelist_revision
May 8, 2026
Merged

[19.0][MIG] product_pricelist_revision: Migration to 19.0#2100
OCA-git-bot merged 29 commits into
OCA:19.0from
HeliconiaIO:19.0-mig-product_pricelist_revision

Conversation

@BhaveshHeliconia
Copy link
Copy Markdown
Contributor

No description provided.

@BhaveshHeliconia BhaveshHeliconia mentioned this pull request Oct 15, 2025
73 tasks
@rousseldenis
Copy link
Copy Markdown
Contributor

/ocabot migration product_pricelist_revision

Copy link
Copy Markdown
Contributor

@gdgellatly gdgellatly left a comment

Choose a reason for hiding this comment

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

Functional test on own db plus code review LGTM.

Copy link
Copy Markdown
Contributor

@gdgellatly gdgellatly left a comment

Choose a reason for hiding this comment

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

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.

sergio-teruel and others added 23 commits March 6, 2026 14:12
[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/
@BhaveshHeliconia BhaveshHeliconia force-pushed the 19.0-mig-product_pricelist_revision branch from 1b96722 to 68cfb1b Compare March 6, 2026 08:42
@BhaveshHeliconia
Copy link
Copy Markdown
Contributor Author

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.

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.

@gdgellatly
Copy link
Copy Markdown
Contributor

gdgellatly commented Mar 8, 2026

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.

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.

Copy link
Copy Markdown
Contributor

@CarlosRoca13 CarlosRoca13 left a comment

Choose a reason for hiding this comment

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

Please add this change: #2291

CarlosRoca13 and others added 2 commits May 8, 2026 13:06
…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
@BhaveshHeliconia BhaveshHeliconia force-pushed the 19.0-mig-product_pricelist_revision branch from 68cfb1b to 76d44c4 Compare May 8, 2026 07:46
@OCA-git-bot OCA-git-bot added series:19.0 mod:product_pricelist_revision Module product_pricelist_revision labels May 8, 2026
@BhaveshHeliconia
Copy link
Copy Markdown
Contributor Author

Please add this change: #2291

Done, thanks!

@pedrobaeza
Copy link
Copy Markdown
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Copy Markdown
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 19.0-ocabot-merge-pr-2100-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 1345655 into OCA:19.0 May 8, 2026
5 of 7 checks passed
@OCA-git-bot
Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at 3ae2dd5. Thanks a lot for contributing to OCA. ❤️

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.