Skip to content

Conversation

@marielejeune
Copy link
Contributor

@marielejeune marielejeune commented Jun 2, 2025

Migration from version 15.0

Addon used since v14 without issue -> change development status to Beta.

Adding unit tests to make codecov succeed

Depends on #385

@marielejeune marielejeune force-pushed the 18.0-mig-product_variant_configurator_manual_creation branch 3 times, most recently from cfd06c4 to 75ab49a Compare June 3, 2025 07:28
@marielejeune marielejeune marked this pull request as ready for review June 3, 2025 07:31
@marielejeune marielejeune force-pushed the 18.0-mig-product_variant_configurator_manual_creation branch from 8694a00 to dc040b7 Compare June 3, 2025 09:23
@marielejeune marielejeune force-pushed the 18.0-mig-product_variant_configurator_manual_creation branch from dc040b7 to 3ce56e8 Compare September 22, 2025 06:34
@marielejeune
Copy link
Contributor Author

marielejeune commented Sep 22, 2025

As for now this PR depends on #404. It could depend on #409 but this PR looks abandoned too...

@marielejeune marielejeune force-pushed the 18.0-mig-product_variant_configurator_manual_creation branch from 3ce56e8 to a097094 Compare September 30, 2025 07:46
@marielejeune
Copy link
Contributor Author

Changed dependency to #385 as it is the oldest and more mature PR.

@marielejeune marielejeune force-pushed the 18.0-mig-product_variant_configurator_manual_creation branch 5 times, most recently from 2d51a7f to 7ebf033 Compare September 30, 2025 09:34
@marielejeune
Copy link
Contributor Author

@CILC98 I've fixed your suggestion and added a commit for improving unit tests and make codecov succeed. Can you re-review please?

@marielejeune marielejeune requested a review from CILC98 September 30, 2025 09:37
@CILC98
Copy link

CILC98 commented Oct 1, 2025

@marielejeune Let's hope change proposal #385 is finalized. Does that sound okay to you?

@marielejeune
Copy link
Contributor Author

@marielejeune Let's hope change proposal #385 is finalized. Does that sound okay to you?

If Pedro's comments are addressed it should be ok and almost ready to merge I think

@rousseldenis
Copy link

/ocabot migration product_variant_configurator_manual_creation

@marielejeune marielejeune force-pushed the 18.0-mig-product_variant_configurator_manual_creation branch from 7ebf033 to add1bde Compare October 15, 2025 15:48
@marielejeune
Copy link
Contributor Author

@CILC98 The product_variant_configurator module was merged. I've rebased this PR. Could you please update your review? We're almost there!

if current_variants_to_create:
variants_to_show |= product_model.create(current_variants_to_create)
if variants_to_show:
action = self.env.ref("product.product_variant_action").read()[0]
Copy link

Choose a reason for hiding this comment

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

Suggested change
action = self.env.ref("product.product_variant_action").read()[0]
action = self.env.ref("product.product_variant_action").sudo().read()[0]

Copy link

Choose a reason for hiding this comment

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

@marielejeune Only the system administrator can read this model.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Right, changed

@marielejeune marielejeune force-pushed the 18.0-mig-product_variant_configurator_manual_creation branch from add1bde to 0d49d80 Compare October 17, 2025 07:04
JasminSForgeFlow and others added 21 commits January 5, 2026 10:48
…ttier

[MIG] product_variant_configurator_manual_creation: Migration to 14.0
Currently translated at 100.0% (22 of 22 strings)

Translation: product-variant-15.0/product-variant-15.0-product_variant_configurator_manual_creation
Translate-URL: https://translation.odoo-community.org/projects/product-variant-15-0/product-variant-15-0-product_variant_configurator_manual_creation/es/
Currently translated at 100.0% (22 of 22 strings)

Translation: product-variant-15.0/product-variant-15.0-product_variant_configurator_manual_creation
Translate-URL: https://translation.odoo-community.org/projects/product-variant-15-0/product-variant-15-0-product_variant_configurator_manual_creation/it/
@marielejeune marielejeune force-pushed the 18.0-mig-product_variant_configurator_manual_creation branch from 1e6668c to cbd1432 Compare January 5, 2026 09:52
@marielejeune
Copy link
Contributor Author

Hi @StefanRijnhart
Even if I agree with Denis (to me we should avoid adding required fields without a default value if no logic asks it),
I've set product_variant_name as a rebel module so that we can merge this PR (which I rebased on 18.0).

@StefanRijnhart
Copy link
Member

Yes, agreed that this is a more viable strategy in the long turn. Thanks for implementing the workaround!

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 18.0-ocabot-merge-pr-406-by-StefanRijnhart-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit f4664c1 into OCA:18.0 Jan 5, 2026
9 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at c0a1126. 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.