-
-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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,dashboard,js-sdk,medusa,types): support Fulfillment Options #10622
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
6 Skipped Deployments
|
|
…shipping method create
30d0243
to
45869f7
Compare
packages/medusa/src/api/admin/fulfillment-providers/[id]/options/route.ts
Show resolved
Hide resolved
...-service-zone-shipping-option-edit/components/edit-region-form/edit-shipping-option-form.tsx
Outdated
Show resolved
Hide resolved
...pping-option-create/components/create-shipping-options-form/create-shipping-options-form.tsx
Show resolved
Hide resolved
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.
Overall LGTM, one final comment.
@kasperkristensen, if you could find time to glance over the UI, that would be great :)
packages/core/core-flows/src/cart/workflows/add-shipping-method-to-cart.ts
Outdated
Show resolved
Hide resolved
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.
UI LGTM, just one small thing to fix
...ption-create/components/create-shipping-options-form/create-shipping-option-details-form.tsx
Outdated
Show resolved
Hide resolved
…ptions (medusajs#10622) **What** - add a list point for fetching fulfillment options for a provider - add FO support on SO create & update on dashboard - pass `cart` and `stockLocation` to `validateFufillmentData` context --- CLOSES CMRC-789 CLOSES CMRC-790
What
cart
andstockLocation
tovalidateFufillmentData
contextCLOSES CMRC-789
CLOSES CMRC-790