-
Notifications
You must be signed in to change notification settings - Fork 2.2k
Add updateOrderFulfillmentGroup mutation #5020
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
Conversation
Signed-off-by: Eric Dobbertin <[email protected]>
Signed-off-by: Eric Dobbertin <[email protected]>
Signed-off-by: Eric Dobbertin <[email protected]>
|
@kieckhafer I add |
kieckhafer
left a comment
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.
Looks good, one quick question but good to approve.
| orderId, | ||
| orderFulfillmentGroupId, | ||
| status, | ||
| tracking |
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.
See that you noted trackingUrl is not updated here yet. Should we go ahead and add it already to keep this in sync with what the other mutation can do?
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 rather wait until we actually add a field in UI for it and need it here.
Signed-off-by: Erik Kieckhafer <[email protected]>
Part of #4999
Impact: minor
Type: feature
Changes
A new
updateOrderFulfillmentGroupmutation allows you to change thetrackingandstatusfields of an order fulfillment group after the order has been placed.The operator UI now uses this mutation for the
trackingupdate only. Changing the operator UI to use GraphQL for group status updates will be done in a later PR.updateOrderFulfillmentGroupcan be called internally to synchronize from an external system by settingcontext.isInternalCalltotrueupdateOrderFulfillmentGroupcan be called through GraphQL by any user with "orders" permission for the shop that owns the orderupdateOrderFulfillmentGroupGraphQL mutationBreaking changes
None
Testing