← Back to team overview

openerp-community-reviewer team mailing list archive

Re: [Merge] lp:~anybox/openerp-product-variant/7.0 into lp:openerp-product-variant

 

Review: Approve

tested and approved.

Just sad that we double worked on that one https://code.launchpad.net/~akretion-team/openerp-product-variant/migr-v7-wip/+merge/148852
But I guess you tested that one more than me (I got no review sadly). Removing the older proposal from Alexis and me then.


Also, I confirm that we want to create a new revamped variant module that will be more compatible with product_custom_attributes. But we think that the best is to let people choose: either use that legacy product_variant_multi module migrated (specially handy if you are migrating and if it works for you) or switch to the new version that will set the bar higher, but only if you can afford switching.

When you keep things like product configurator in mind, it's important that the variant module is compatible with product_custom_attributes. Indeed, you can very much see variants as just a special case of configurable product as a pre-defined combo for which you want to manage the stock instead of just being an "on demand" combo for just one specific order. If concepts are compatible then it's easier to build product configurator interfaces.
-- 
https://code.launchpad.net/~anybox/openerp-product-variant/7.0/+merge/183609
Your team Product Core Editors is subscribed to branch lp:openerp-product-variant.