-
Notifications
You must be signed in to change notification settings - Fork 35
feat: pending order status (PEXP-1358) #209
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
base: master
Are you sure you want to change the base?
Conversation
|
Hi! I'm VTEX IO CI/CD Bot and I'll be helping you to publish your app! 🤖 Please select which version do you want to release:
And then you just need to merge your PR when you are ready! There is no need to create a release commit/tag.
|
|
Beep boop 🤖 I noticed you didn't make any changes at the
In order to keep track, I'll create an issue if you decide now is not a good time
|
What is the purpose of this pull request?
Warning
This PR depends on and should be merged after https://github.com/vtex/order-placed-graphql/pull/40
The aim here is to add a pending order confirmation message for the scenario where the API returns a
202status code for the/ordersendpoint. Check the above PR for more info.Now, we check for the new
OrderGroup.statusfield, which can bependingorconfirmed, and then display the proper confirmation message, based on this figma spec.How should this be manually tested?
Pending order status
Confirmed order status
22210060as the postal codeScreenshots or example usage
Types of changes