[19.0][MIG] website_sale_wishlist_keep: Migration to version 19.0 - #1291
[19.0][MIG] website_sale_wishlist_keep: Migration to version 19.0#1291cristina-hidalgo-tecnativa wants to merge 20 commits into
Conversation
…ishlist when we add to cart
OCA-git-bot
added
series:19.0
mod:website_sale_wishlist_keep
|
/ocabot migration website_sale_wishlist_keep |
e184fa4 to
8384683
Compare
Andrii9090-tecnativa
left a comment
Andrii9090-tecnativa
left a comment
There was a problem hiding this comment.
LGTM, test on runboat 👍
pilarvargas-tecnativa
left a comment
pilarvargas-tecnativa
left a comment
There was a problem hiding this comment.
Please restore the Odoo option in this module to how it was in previous versions
|
Use odoo/odoo@b414047 as reference |
8384683 to
9f39e29
Compare
adasatorres-tecnativa
left a comment
adasatorres-tecnativa
left a comment
There was a problem hiding this comment.
LGTM, Code review and functionality testing on Runboat.
pilarvargas-tecnativa
left a comment
pilarvargas-tecnativa
left a comment
There was a problem hiding this comment.
Could you update the PR description? The current implementation restores the #b2b_wish checkbox, so the data-keep-wishlist explanation is outdated.
9f39e29 to
cf4538c
Compare
pilarvargas-tecnativa
left a comment
pilarvargas-tecnativa
left a comment
There was a problem hiding this comment.
The option needs to be moved in the editor, as it's disrupting the product design.
cf4538c to
d837366
Compare
Modified. |
14 participants
Odoo 19 removed the
#b2b_wishcheckbox and its JS logic fromwebsite_sale_wishlist; adding a product to cart from the wishlist now always removes it, with no built-in option. This module restores that checkbox and its original behavior:#b2b_wishcheckbox on the wishlist page (removed upstream in b414047).ProductWishlistreproducing the removed core condition (if (quantity > 0 && !checkbox.checked) removeProduct()).customize_showview that pre-checks it by default, now exposed from the new Style tab (Wishlist Page section) instead of the old Customize tab.@Tecnativa TT64171
@pedrobaeza @pilarvargas-tecnativa