-
-
Notifications
You must be signed in to change notification settings - Fork 308
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
[14.0] Add edi_sale_oca + edi_sale_ubl_oca #759
Conversation
7d45d64
to
a3209eb
Compare
28fc5e8
to
e2d9b9c
Compare
119b59e
to
c2ac8c0
Compare
Additionally, avoid recomputing states for all lines when we know which lines have changed.
Several lines' state make the order not satisfied.
Lines w/ no qty must not be excluded a priori
566f105
to
1b161c2
Compare
Splitting edi_sale_oca + endpoint to #831 and removing dependency on |
Hi @simahawk , should we need to move |
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
qty = self.product_uom_qty | ||
if not qty: | ||
return self.order_id.EDI_STATE_ORDER_LINE_NOT_ACCEPTED | ||
if qty > vals["product_uom_qty"] or qty < vals["product_uom_qty"]: |
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.
TODO: consider comparing available qty. This can come from sale_stock_available_to_promise_release
so we need an hook for a glue module just for the qty comparison.
Provide full cycle for inbound Order + outbound OrderResponse.
These modules are now deprecated in favour of a more organic
edi_sale_oca
+edi_sale_ubl_oca
:Depends on:
TODO: