Skip to content

Conversation

@pilarvargas-tecnativa
Copy link
Contributor

@pilarvargas-tecnativa pilarvargas-tecnativa commented Jun 26, 2025

@Tecnativa TT54539

@CarlosRoca13 @victoralmau please review

@pedrobaeza
Copy link
Member

/ocabot migration website_sale_product_item_cart_custom_qty

Check the CI

@OCA-git-bot OCA-git-bot added this to the 18.0 milestone Jun 26, 2025
@OCA-git-bot OCA-git-bot mentioned this pull request Jun 26, 2025
40 tasks
dessanhemrayev and others added 3 commits June 27, 2025 14:39
Add a setting in /shop page > customize: Always display qty buttons
Qty and add to cart buttons are always visible (not only on rollover)
To improve the visualization, move the buttons below the image so that they do not overlap it, now it occupies about 20% of the image height.
@pilarvargas-tecnativa pilarvargas-tecnativa force-pushed the 18.0-mig-website_sale_product_item_cart_custom_qty branch from c16e588 to eced34c Compare June 27, 2025 12:48
Copy link
Contributor

@CarlosRoca13 CarlosRoca13 left a comment

Choose a reason for hiding this comment

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

LGTM 👍 Code and functional review

@pedrobaeza pedrobaeza requested a review from victoralmau June 30, 2025 11:01
Copy link
Member

@victoralmau victoralmau left a comment

Choose a reason for hiding this comment

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

Functional review OK

@pedrobaeza
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

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

@OCA-git-bot OCA-git-bot merged commit 0050a96 into OCA:18.0 Jun 30, 2025
7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 1159aa5. Thanks a lot for contributing to OCA. ❤️

@pedrobaeza pedrobaeza deleted the 18.0-mig-website_sale_product_item_cart_custom_qty branch June 30, 2025 12:20
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.

9 participants