Merge lp://staging/~camptocamp/product-extra-addons/extra-trunk-fix-1017960 into lp://staging/product-extra-addons
Proposed by
Alexandre Fayolle - camptocamp
| Status: | Merged |
|---|---|
| Approved by: | Sébastien BEAU - http://www.akretion.com |
| Approved revision: | 48 |
| Merged at revision: | 61 |
| Proposed branch: | lp://staging/~camptocamp/product-extra-addons/extra-trunk-fix-1017960 |
| Merge into: | lp://staging/product-extra-addons |
| Diff against target: |
129 lines (+36/-14) 2 files modified
product_images_olbs/product.py (+21/-7) product_images_olbs/product_images.py (+15/-7) |
| To merge this branch: | bzr merge lp://staging/~camptocamp/product-extra-addons/extra-trunk-fix-1017960 |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Sébastien BEAU - http://www.akretion.com | Pending | ||
| Guewen Baconnier @ Camptocamp | Pending | ||
| Nicolas Bessi - Camptocamp | Pending | ||
| Raphaël Valyi - http://www.akretion.com | Pending | ||
|
Review via email:
|
|||
To post a comment you must log in.
Revision history for this message
| Sébastien BEAU - http://www.akretion.com (sebastien.beau) wrote | # |
Revision history for this message
| Alexandre Fayolle - camptocamp (alexandre-fayolle-c2c) wrote | # |
Revision history for this message
| Sébastien BEAU - http://www.akretion.com (sebastien.beau) wrote | # |
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
Hi Alex.
I don't know if the best solution will be to not copy the image. Indeed when you duplicate a product, I am not sure that the customer want to keep the same image, so maybe it's simplier to not copy the image_ids and set it to false.
What do you think?
Thank you