-
-
Notifications
You must be signed in to change notification settings - Fork 482
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[13.0] [MIG] website_sale_wishlist_keep: Migration to v13.0 #496
[13.0] [MIG] website_sale_wishlist_keep: Migration to v13.0 #496
Conversation
…ishlist when we add to cart
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code and functional review: 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code and functional review 👍
Just a small suggestion to the docs, but nothing blocking
#. Active Customize > Keep items on wishlist | ||
#. You can see the option checked allways that you go to the wishlist. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
#. Active Customize > Keep items on wishlist | |
#. You can see the option checked allways that you go to the wishlist. | |
#. You can see the "Add product to my cart but keep it in my wishlist" option checked by default |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The first step that you suggest to delete is needed, but probably someone do it before you
a1d2ecd
to
94d2a78
Compare
94d2a78
to
0ea74a3
Compare
This PR has the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Functional review
/ocabot merge nobump |
This PR looks fantastic, let's merge it! |
Congratulations, your PR was merged at 881985b. Thanks a lot for contributing to OCA. ❤️ |
cc @Tecnativa TT27485
Please @joao-p-marques @ernestotejeda review this