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

Checkout with paypal takes ~10s on production ... #101

Closed
NinoSkopac opened this issue Mar 15, 2017 · 4 comments · Fixed by #103
Closed

Checkout with paypal takes ~10s on production ... #101

NinoSkopac opened this issue Mar 15, 2017 · 4 comments · Fixed by #103

Comments

@NinoSkopac
Copy link

NinoSkopac commented Mar 15, 2017

... and there is no indicator right now that something's going on, so the user will surely think it ain't working and drop it, just to be billed anyways. demo: https://read2me.online/ (you have to be logged in)

@NinoSkopac NinoSkopac changed the title Checkout with paypal takes ~10 on production ... Checkout with paypal takes ~10s on production ... Mar 15, 2017
@crookedneighbor
Copy link
Contributor

Thanks for reporting this.

When we switch to use the paypal-checkout component with checkout.js this should be solved, as checkout.js will provide UI to indicate that the transaction is still in progress.

@NinoSkopac
Copy link
Author

Awesome. Any ETA on that? Will it break BC?

@crookedneighbor
Copy link
Contributor

I'm working on the PR right now. We're waiting for a change on the Braintree gateway before we can merge it in. The one required change for you will be that you will need to create and link a PayPal sandbox account in your Braintree GW.

I do not have an ETA yet.

@crookedneighbor
Copy link
Contributor

You may want to consider passing intent: 'sale' in your PayPal config. That should speed up the process a bit as it does an auth and capture in one step instead of doing that work on the server after the fact.

http://braintree.github.io/braintree-web/current/PayPal.html#tokenize

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

Successfully merging a pull request may close this issue.

2 participants