-
-
Notifications
You must be signed in to change notification settings - Fork 704
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
8.0 port module stock_reserve_sale #14
Conversation
i think the build is red because of OCA/maintainer-quality-tools#91 👍 |
Now the build is still red, but that's just because of a minor lint error. I'll fix it when merging. |
I realised I had the right to fix directly. We're green now. Thanks! |
for line in lines: | ||
if not line.is_stock_reservable: | ||
continue | ||
vals = self._prepare_stock_reservation(line)[0] |
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.
you can remove the [0] by changing the decorator of the function from api.one
to api.multi
(use self.ensure_one()
at the beginning of the method if needed)
…6. setting base string as unicode didn't solve it
d173695
to
0fba1c1
Compare
@gurneyalex I addressed your remarks. I also rebased to be able to test, to bring #8 in. Thanks! |
👍 |
👍 Thanks for the fixes @lepistone |
8.0 port module stock_reserve_sale
…l_procurement_setup-dro [10.0][FIX] stock orderpoint manual procurement: Add setup
Includes re-factoring with new API
This PR depends on #8 which is merged 😸