-
Notifications
You must be signed in to change notification settings - Fork 0
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
[pull] develop from medusajs:develop #2
Conversation
Reviewer's Guide by SourceryThis pull request introduces several significant changes across the Medusa codebase, primarily focusing on cart workflows, pricing context handling, and various module improvements. The changes include updates to payment processing, shipping calculations, and several bug fixes. Sequence diagram for addToCartWorkflowsequenceDiagram
participant User
participant CartService
participant WorkflowManager
User->>CartService: Add item to cart
CartService->>WorkflowManager: run addToCartWorkflow
WorkflowManager->>CartService: validateCartStep
WorkflowManager->>CartService: useQueryGraphStep to get cart
WorkflowManager->>CartService: transform to get variantIds
WorkflowManager->>CartService: useRemoteQueryStep to fetch variants
WorkflowManager->>CartService: prepareLineItemData
WorkflowManager->>CartService: confirmVariantInventoryWorkflow
WorkflowManager->>CartService: createLineItemsStep
WorkflowManager->>CartService: updateLineItemsStep
WorkflowManager->>CartService: refreshCartItemsWorkflow
WorkflowManager->>CartService: emitEventStep
CartService->>User: Return updated cart
Sequence diagram for calculateShippingOptionsPricesWorkflowsequenceDiagram
participant CartService
participant WorkflowManager
CartService->>WorkflowManager: run calculateShippingOptionsPricesWorkflow
WorkflowManager->>CartService: useQueryGraphStep for shippingOptionsQuery
WorkflowManager->>CartService: useQueryGraphStep for cartQuery
WorkflowManager->>CartService: transform to get fulfillmentSetId
WorkflowManager->>CartService: useQueryGraphStep for locationFulfillmentSetQuery
WorkflowManager->>CartService: transform to get locationIds
WorkflowManager->>CartService: useQueryGraphStep for locationQuery
WorkflowManager->>CartService: transform to prepare data
WorkflowManager->>CartService: Return calculated shipping options prices
Class diagram for FulfillmentModuleService modificationsclassDiagram
class FulfillmentModuleService {
+ deleteFulfillment(string id, Context sharedContext)
}
class Context {
<<imported>>
}
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
Important Review skippedBot user detected. To trigger a single review, invoke the You can disable this status message by setting the 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
See Commits and Changes for more details.
Created by pull[bot] (v2.0.0-alpha.1)
Can you help keep this open source service alive? 💖 Please sponsor : )
Summary by Sourcery
Enhance the Medusa project with a new workflow for listing shipping options with pricing, and refactor the Stripe integration in the Next.js Starter to support multiple payment methods using the Stripe Payment Element. Update package versions and documentation, and add tests for the new features.
New Features:
Enhancements:
Build:
Documentation:
Tests: