feat(core-flows): manage inventory reservation of claims and exchanges #8448
Conversation
…re/order-exchange-2
carlos-r-l-rodrigues
requested review from
riqwan,
adrien2p and
olivermrbl
and removed request for
a team
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
6 Skipped Deployments
|
|
….com/medusajs/medusa into feat/claim-exchange-manage-inventory
carlos-r-l-rodrigues
changed the base branch from
chore/order-exchange-2
to
develop
…t/claim-exchange-manage-inventory
….com/medusajs/medusa into feat/claim-exchange-manage-inventory
| when({ input }, ({ input }) => { | ||
| return !input.skipInventoryCheck | ||
| }).then(() => { | ||
| confirmInventoryStep(confirmInventoryInput) | ||
| }) |
There was a problem hiding this comment.
question: Does it make sense for this check to live inside the confirmVariantInventoryWorkflow? Shouldn't it be outside of it when you consume it?
There was a problem hiding this comment.
Well, this is a good point.
I think it makes sense to separate in a util the portion to format the input to confirm/reserve inventory, from the inventory check.
I will open a follow up PR organizing that.
No description provided.