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

Order / Auth / Capture #487

Closed
angelleye opened this issue Jun 21, 2016 · 8 comments
Closed

Order / Auth / Capture #487

angelleye opened this issue Jun 21, 2016 · 8 comments
Assignees
Milestone

Comments

@angelleye
Copy link
Collaborator

I ran an order with a total of $42.69. When I go to Authorize a payment against the order, though, it shows a message about making sure I enter an amount less than $49.09.

Shouldn't those numbers match? Why would it let me authorize 49.09 if the order total was 42.69?

image

@angelleye angelleye added this to the 1.2.2 milestone Jun 21, 2016
@angelleye
Copy link
Collaborator Author

Also, we need to make sure the new setting for Order / Auth / Sale in the plugin defaults to Sale even if the settings page has not fully been saved yet. Seems that some people are having "orders" go through instead of "sale" when they never switched it to that.

@kcppdevelopers
Copy link
Contributor

In Express Checkout Authorization user can capture less than the original authorization amount, the full amount, or up to 115% of or $75 more than the original authorization, whichever is less.

order total was 42.69, user can capture 115% of 42.69 = 49.09

@angelleye
Copy link
Collaborator Author

Are we sure that's the same with an order, though? Did you test that specifically? I know that's true for original Authorizations, but with an order where Authorizations are happening against it I'm not sure that's true.

@kcppdevelopers
Copy link
Contributor

Yes I am sure.

@kcppdevelopers
Copy link
Contributor

I have already test this functionality and it's working fine for me.

@angelleye
Copy link
Collaborator Author

Ok, thanks for checking.

@angelleye
Copy link
Collaborator Author

Ah, well, I shouldn't close it yet because I had that other comment about the default setting. Have you looked into that already?

@angelleye angelleye reopened this Jun 21, 2016
@kcppdevelopers
Copy link
Contributor

I have already applied changes in issue #486

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

No branches or pull requests

1 participant