-
-
Notifications
You must be signed in to change notification settings - Fork 498
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
[MIG] website_sale_stock_available: Migration to 16.0 #708
[MIG] website_sale_stock_available: Migration to 16.0 #708
Conversation
f5f31a0
to
dc9e1a2
Compare
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.
Take a look at #714 You need to adapt the module to Odoo core changes
Hello @chienandalu , I have made the changes as per #714 , can you please review and approve the same |
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.
I'd prefer you to use the now merged #714 as your main migration path as there are several formal improvements (updated readme, tests, etc). Hopefully there will be not much to do after that for the 16.0 migration
…ties With previous one, in case of no stock, it's not changed to potential qty.
When the `virtual_available` is less than the `immediately_usable_qty`, we'll have a blocking situation when the order is paid if we could add more quantity to the order than the virtual one.
Currently translated at 100.0% (6 of 6 strings) Translation: e-commerce-14.0/e-commerce-14.0-website_sale_stock_available Translate-URL: https://translation.odoo-community.org/projects/e-commerce-14-0/e-commerce-14-0-website_sale_stock_available/es_AR/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: e-commerce-15.0/e-commerce-15.0-website_sale_stock_available Translate-URL: https://translation.odoo-community.org/projects/e-commerce-15-0/e-commerce-15-0-website_sale_stock_available/
81472c5
to
9bd462c
Compare
@chienandalu , updated based on #714 |
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.
Thanks! 🙂 👍
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.
/ocabot migration website_sale_stock_available
/ocabot merge nobump
This PR looks fantastic, let's merge it! |
This PR has the |
Congratulations, your PR was merged at b2bbc48. Thanks a lot for contributing to OCA. ❤️ |
No description provided.