Skip to content
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.

Sign up for GitHub

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

Jump to bottom

feat: async checkout polling #5164

Merged
merged 10 commits into from
Feb 21, 2024
Merged

feat: async checkout polling #5164

merged 10 commits into from
Feb 21, 2024

Conversation

mcstover
Copy link
Collaborator

@mcstover mcstover commented Feb 13, 2024 •
edited
Loading

Started a fresh branch off of main and cherry-picked the 2 outstanding original commits.

CURRENT STATE:

  • Happy path async checkout works for both KivaCredit and Braintree checkouts, I was able to observe multiple network calls to the checkoutStatus mutation and transition to the thanks page after hitting the "COMPLETED" status.

@mcstover mcstover requested review from emuvente, dyersituations and a team February 13, 2024 22:26
@mcstover mcstover changed the title Feat async checkout polling feat: async checkout polling Feb 13, 2024
@xumical xumical mentioned this pull request Feb 18, 2024
Open
@mcstover mcstover mentioned this pull request Feb 21, 2024
Closed
@mcstover mcstover merged commit 31c977c into main Feb 21, 2024
5 checks passed
@mcstover mcstover deleted the feat-async-checkout-polling branch February 21, 2024 19:00
@kiva-robot
Copy link
Collaborator

🎉 This PR is included in version 2.731.0-rc.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@kiva-robot
Copy link
Collaborator

🎉 This PR is included in version 2.731.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@xumical xumical mentioned this pull request Apr 16, 2024
Open
@xumical xumical mentioned this pull request Apr 29, 2024
Open
@xumical xumical mentioned this pull request May 9, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dyersituations dyersituations dyersituations approved these changes

@emuvente emuvente Awaiting requested review from emuvente

Assignees
No one assigned
Labels
released on @staging released
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@mcstover @kiva-robot @dyersituations