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

Express Checkout - Skip Review vs. Terms and Conditions #471

Closed
angelleye opened this Issue May 26, 2016 · 2 comments

Comments

Projects
None yet
2 participants
@angelleye
Owner

angelleye commented May 26, 2016

As it stands now we have an option for "skip review", but if you have a terms and conditions page set in WooCommerce then this overrides the skip review and displays that review page anyway so that the user is forced to accept the terms.

We have some user feedback (https://wordpress.org/support/topic/skip-final-review-option-enabled-in-express-checkout-but-not-working?replies=3#post-8450341) where they require terms and conditions for other payment methods, but they don't want it enabled for Express Checkout, and they would rather skip details.

We should go ahead and add an option into the Express Checkout settings to allow for people to set this up however they want.

For example, if they check the box to enable the skip payment options we could have some ajax check to see if they have a terms and conditions page enabled. If they do, we could display a message that it would override and force the review page anyway, but then we can provide the option there to disable T&C for Express Checkout only so that the review page could indeed be skipped.

@angelleye angelleye added this to the 1.2.1 milestone May 26, 2016

@angelleye angelleye modified the milestones: 1.2.1, 1.2.2 Jun 18, 2016

@angelleye angelleye modified the milestones: 1.2.2, 1.2.3, 1.2.4 Jun 20, 2016

@angelleye angelleye modified the milestones: 1.2.4, 1.2.5 Jun 29, 2016

@kcppdevelopers kcppdevelopers modified the milestones: 1.3, 1.2.4 Jul 28, 2016

@angelleye angelleye modified the milestones: 1.3.0, 1.2.5 Aug 16, 2016

@angelleye

This comment has been minimized.

Show comment
Hide comment
@angelleye

angelleye Nov 16, 2016

Owner

@kcppdevelopers I'm getting a small merge conflict on this one as well. Please take a look and merge into dev for me.

Owner

angelleye commented Nov 16, 2016

@kcppdevelopers I'm getting a small merge conflict on this one as well. Please take a look and merge into dev for me.

@kcppdevelopers

This comment has been minimized.

Show comment
Hide comment
@kcppdevelopers

kcppdevelopers Nov 16, 2016

Collaborator

I have merged into dev branch.

Collaborator

kcppdevelopers commented Nov 16, 2016

I have merged into dev branch.

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