-
-
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
docs: fix cart completion in subscriptions recipe #10021
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Skipped Deployment
|
} | ||
}) | ||
|
||
const order = useRemoteQueryStep({ |
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.
thought: Should we use useQueryStep
so we slowly migrate to use the query.graph
everywhere?
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.
Oh I wasn't aware of it, we definitely should. Will do it in a separate PR 👍
Fix subscriptions recipe to retrieve the order's details after completion with
completeCartWorkflow
(since the workflow returns only the ID now).