-
Notifications
You must be signed in to change notification settings - Fork 66
flow_config.user_action cannot be set to "Continue" #44
Comments
Hey @Connum, The If this doesn't work for you let me know! I'll look into it. Cheers! |
Thanks for the quick response! |
Hey @Connum, Did you get it to work? |
Unfortunately, I still haven't been able to test it due to the API error. Even tried a new key, but to no avail. But I hope this will be sorted out soon! |
Hi @khoaNguyen96, |
One thing that I still don't understand though is that the payment is automatically being completed after authorization, without the user having to confirm it on the page again. This seems to be triggered by the component? Shouldn't there be a check in "onAuthorize" for the commit prop? |
Hey @Connum, I never used it without Cheers! P.S. Thanks for the PR I'll take a look at it as soon as I can. 👍 |
I imported and used this code and provided sandbox client ID.But not working.can you please help me how to get responses while doing payment |
hey @bhaskararao-nyros, The event names were changed recently. https://github.com/khoanguyen96/vue-paypal-checkout#events-fired-by-the-simple-paypal-component What version of vue-paypal-checkout do you have installed? |
Hi there,
this might not be the component's fault, but as the docs state that a "valid object with the Experience options" can be passed, I'm posting this here in the hope that someone has an idea.
This is what my experienceOptions look like:
However, as soon as I set the user_action to 'Continue' (also tried small-caps 'continue'), I get an error from the API:
Any idea why I shouldn't be able to set the user_action to anything other than 'commit'? Maybe a combination with another (default) setting that prevents it? I don't want to use the "Pay now" approach, because I fear that a tech-savvy user could tamper with the cart amount, so I want to get the Payment ID and check that server-side before making the actual purchase. And as the component doesn't support Advanced Integration yet (see #36), I think there's no other option.
Any help appreciated.
The text was updated successfully, but these errors were encountered: