-
Notifications
You must be signed in to change notification settings - Fork 40
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
Implemented: dropdown for other shipments in a order in order details page (#358) #364
Conversation
src/services/OrderService.ts
Outdated
@@ -258,8 +258,52 @@ const getShippingPhoneNumber = async (orderId: string): Promise<any> => { | |||
return phoneNumber | |||
} | |||
|
|||
const findOrderShipGroup = async (query: any): Promise<any> => { | |||
return api({ | |||
// TODO: We can replace this with any API |
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.
Remove this TODO.
src/store/modules/order/actions.ts
Outdated
|
||
const params = { | ||
groupBy: 'shipGroupSeqId', | ||
'shipGroupSeqId': '[* TO *]', // check to ignore all those records for which shipGroupSeqId is not present, as in case of kit comp we does not get shipGroupSeqId on some items |
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.
Update comment.
src/store/modules/order/actions.ts
Outdated
resp = await OrderService.findOrderShipGroup(orderQueryPayload); | ||
|
||
if (resp.status === 200 && !hasError(resp) && resp.data.grouped?.shipGroupSeqId.matches > 0) { | ||
total = resp.data.grouped.shipGroupSeqId.ngroups |
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.
Unused variable.
Related Issues
Closes #358
Short Description and Why It's Useful
Implemented code to display other shipments in a order in order details page.
Screenshots of Visual Changes before/after (If There Are Any)
Contribution and Currently Important Rules Acceptance