Skip to content

Conversation

@itariv
Copy link
Collaborator

@itariv itariv commented Oct 30, 2025

No description provided.

@itariv itariv requested a review from mwestrjs October 30, 2025 13:06
@medusajs
Copy link

medusajs bot commented Oct 30, 2025

Name Deployment Status Preview Link Timestamp
Prokitchen 🔴 Deployment Failed Preview Thu, 30 Oct 2025 13:11:47 GMT

@medusajs
Copy link

medusajs bot commented Oct 30, 2025

Name Deployment Status Preview Link Timestamp
Mercur-Testing ✅ Ready Preview Thu, 30 Oct 2025 13:15:00 GMT

@medusajs
Copy link

medusajs bot commented Oct 30, 2025

Name Deployment Status Preview Link Timestamp
Mercur ✅ Ready Preview Thu, 30 Oct 2025 13:13:48 GMT

@itariv itariv added the don't merge do not merge (yet) label Oct 30, 2025
fields: ["id", "reserved_quantity"],
});

if (!result.data.length) {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wrap in try catch (not sure if needed) and add to query.graph

{ throwIfKeyNotFound: true }

fields: ["id", "reserved_quantity"],
})

if (!result.data.length) {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wrap in try catch (not sure if needed) and add to query.graph

{ throwIfKeyNotFound: true }

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

don't merge do not merge (yet)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants