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

Express Checkout - No Shipping Required Adjustment #831

Closed
angelleye opened this issue Aug 11, 2017 · 1 comment
Closed

Express Checkout - No Shipping Required Adjustment #831

angelleye opened this issue Aug 11, 2017 · 1 comment
Assignees
Milestone

Comments

@angelleye
Copy link
Collaborator

If No Shipping Required is set for a product then we aren't getting any billing or shipping info at all in the WC order.

This makes sense to have no shipping address, but the billing address should at least have the person's name / company name filled in, which would be available in the GECD response even without a shipping address.

@angelleye angelleye added this to the 1.5.0 milestone Aug 11, 2017
@angelleye
Copy link
Collaborator Author

angelleye commented Aug 11, 2017

Take a look at these order comparisons:

image

7144 - This order was done via Express Checkout. No Shipping Required was NOT enabled, so we can see all of the data for billing/shipping is populated, and the order came in with the name Steve Pfister.

7141 - This order was placed via Payments Pro using the WC checkout page. For this product No Shipping Required was enabled, but the billing info was filled out on the WC checkout page. As such, we see what we expect, which is that the order has billing info but not shipping info. Again, it correctly added Michael Vernon as the name on the order.

7129 - This is where the problem is. This order was placed with Express Checkout for a product that had No Shipping Required enabled. Not only do we end up with nothing entered for billing and shipping, but the name on the order also came through as Guest. We should at least have the person's name / company name included for billing, and the order should be associated with the name instead of Guest.

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

1 participant