Skip to content

Conversation

@pilarvargas-tecnativa
Copy link
Contributor

@Tecnativa TT57286

@pedrobaeza @CarlosRoca13 please review

CarlosRoca13 and others added 13 commits December 17, 2025 22:46
…ite_sale_stock_available

When website_sale_stock_available is installed, the grid view is showing
different allowed quantities than when we access to the products sheet.

With this changes we add compatibility with this module and if
stock_available_immediately is installed we will get same value.

TT38689
As the value of available_threshold was set by default if some
module decrease this default, the test will fail, because the
text rendered will be "In stock" instead of "5.000 Units available".
Since the changes introduced in odoo/odoo@a8c73eb
we need to pass to the template of the remaining units to the function formatQuantity. So we
need to pass the function.

The steps to reproduce the error:
1. Edit some product, example Customizable Desk
2. Select the option Show Available Qty and set it to 999999
3. Go to /shop

You will see the error
@pedrobaeza
Copy link
Member

/ocabot migration website_sale_stock_list_preview

@OCA-git-bot OCA-git-bot added this to the 18.0 milestone Dec 20, 2025
@OCA-git-bot OCA-git-bot mentioned this pull request Dec 20, 2025
40 tasks
Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

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

Tested and working!

@pedrobaeza
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 18.0-ocabot-merge-pr-1149-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit eae25cc into OCA:18.0 Dec 22, 2025
7 checks passed
@OCA-git-bot
Copy link
Contributor

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

@pedrobaeza pedrobaeza deleted the 18.0-mig-website_sale_stock_list_preview branch December 22, 2025 14:36
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.

7 participants