-
-
Notifications
You must be signed in to change notification settings - Fork 758
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][FIX] purchase_request: Proper Run Buy implementation #853
Conversation
c23ed73
to
e838542
Compare
e838542
to
351f1d7
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.
LGTM
351f1d7
to
a7067c3
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.
Created to Orderpoints, one for directly create a PO and the other for purchase requests. Run the scheduler. Only the PO was created for the product. No purchase request was created.
For testing I used newly created products and two different vendors for each product. One product had purchase request + buy as routes. The order had just Buy.
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.
👍 Dismiss my previous comment, it works.
off topic: it was something related to multicompany, for some reason the orderpoint was defaulted to Chicago instead of San Francico, even when I was working in San Francisco 🤔
This PR has the |
/ocabot merge patch |
What a great day to merge this nice PR. Let's do it! |
Congratulations, your PR was merged at a287fbb. Thanks a lot for contributing to OCA. ❤️ |
This fix aims to properly run either PR or PO when multiple procurements are thrown.
CC @ForgeFlow @kongrattapong