-
Notifications
You must be signed in to change notification settings - Fork 77
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 - Checkout Page Address(es) #215
Comments
Just remember that if people checkout through the cart page or product detail page then they haven't entered an address yet, so in that case we would not be sending an address in SEC to PayPal so would not need any override. |
One caveat, though, is that if the user is signed into the site or creates an account, they may have a shipping address available from that data. In this case, though, what we should do is send the address in the SEC request to PayPal, but WITHOUT the override. This way it will fill in the address the site has on file for them, but if that address is wrong for any reason the user would still be able to change it to an address they have setup in their PayPal account. |
One more thing here. If the user checks out via the WC checkout page then we've already obtained the shipping address, calculated shipping/tax, etc. As such, we do not need to display the review page that typically shows up when the user is returned from PayPal. Instead, we can treat it as if "skip final review" was enabled even if that option is not enabled. In this scenario it makes sense to go ahead and treat it that way. The logic for handling such a checkout is already built in as long as "skip final review" is enabled, so we could probably just add a little bit of logic around that setting to make sure it gets enabled when the WC checkout page is used. |
I am done with this and push to issue #215 branch for your review. |
I am unable to get this merged due to a merge conflict. Please merge the current release branch into this branch and resolve the conflicts for me. |
Not sure if you're done yet, but I'm still getting a merge conflict when I try to merge this into the release branch. |
I have resolved conflict and push issue-215-B branch into the release branch. |
I may have confused you with something I mentioned in a previous comment. Now, we have 2 separate params:
We do not need both of those parameters. PayPal's SetExpressCheckout documentation shows that the correct parameter name is "addroverride", so that is the only one we need. We just need to make sure that it's getting used when we need it to, and is working correctly with the checkout when we test that scenario. |
I have remove "addressoverride" parameter and push into the release branch. |
Currently, I do not think we are using the ADDRESSOVERRIDE parameter in SetExpressCheckout at all.
This is only necessary if the user checks out through the WooCommerce checkout page instead of using an EC button from the cart page or product page.
When checking out from the WC checkout page the user is typing in their billing and shipping address. As such, we need to include this shipping address in the PayPal SetExpressCheckout request along with the ADDRESSOVERRIDE set to 1/true so that the address used on the WC checkout page is carried out all the way through the rest of checkout and cannot be changed in the PayPal checkout pages.
The text was updated successfully, but these errors were encountered: