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.

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

Improve UX for stripe interactions #105

Open
lilliputten opened this issue Apr 4, 2024 · 1 comment
Open

Improve UX for stripe interactions #105

lilliputten opened this issue Apr 4, 2024 · 1 comment
Assignees

Comments

@lilliputten
Copy link
Collaborator

lilliputten commented Apr 4, 2024

There are two places when the app silently interacts with the stripe api:

  • In dds_registration/views/helpers/start_stripe_payment_intent.py:initiate_stripe_payment, on PaymentIntent creation.
  • On the client side, in src/assets/stripe-init/stripe_payment_intents_support.ts:submitStripeForm, when api requested (via stripe.confirmPayment) before 'success' redirect.

In the 1st case it's possible to return to previous approach, when started the payment and fetched the secret via async ajax request after the payment page has been already opened: this operation requires a noticeable time (up to 1-2 seconds required to open the page).

In the 2nd case, we can display (for example) 'busy' spinner to show that an operation is being processed.

@lilliputten lilliputten self-assigned this Apr 4, 2024
@cmutel
Copy link
Contributor

cmutel commented Apr 5, 2024

In the 2nd case, we can display (for example) 'busy' spinner to show that an operation is being processed.

Yes, let's do that.

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

No branches or pull requests

2 participants