-
-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
feat(core-flows,order,medusa): exchange endpoints #8396
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
6 Skipped Deployments
|
|
…re/order-exchange-2
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.
Lots of stuff in this PR. Can we try to keep them smaller going forward because it's a bit hard to review?
I left a few comment, but I would be OK getting this merged so it doesn't block any UI implementation work.
) | ||
|
||
export const removeItemExchangeActionWorkflowId = "remove-item-exchange-action" | ||
export const removeItemExchangeActionWorkflow = createWorkflow( |
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.
would love to get these steps to be reusable soon 😄
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 will work to clean these similar steps everywhere in Order flows
@@ -0,0 +1,96 @@ | |||
import { |
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.
question: Why do we fetch and return returns
in these endpoints when they are for exchanges?
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.
In practice, inbounds are returns
@@ -0,0 +1,60 @@ | |||
import { orderExchangeRequestItemReturnWorkflow } from "@medusajs/core-flows" |
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.
same question here
@@ -0,0 +1,110 @@ | |||
import { |
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.
Same question here, will not comment more on this anymore if there are other similar endpoints.
Order Exchange: PR 2:n
What:
Next PR will contain inventory management of claims and exchanges.