-
Notifications
You must be signed in to change notification settings - Fork 92
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
Charge method improvement #20
Comments
when it's a soft decline, the status is Suspended hence it won't go into the 'Draft' IF statement when the ORO gets to 'Open' state calling charge method will go into |
Maybe we could add a ReCharge method/param... it would do a GetORO (see if Suspended) instead of Set, and then Confirm, Auth for existing oro's. The Charge method could return to the calling method (which could be a javascript ajax call) to allow the front-end to message the customer easily, before a re-charge. This could potentially be leveraged for auto-handling of declines for asynchronous or charge when ship, where the customer is notified after they've left the site there was an issue with payment. Let's keep in the spirit of the updated SDKs and simplify the integration and prescribe the use cases. Handling declines is needed by all integrations. |
@aaronrichmond I am getting confused here. merchant calls charge method and then the ORO goes to suspended charge fails. |
@shrakuma When talking about sync auth, the workflow is different.
In this situation the ORO will still be suspended and we need to call confirm again, then authorize and capture. @aaronrichmond I would go for the Charge method taking care of this. The big advantage is being able to keep your code untouched and simply call charge again. If we can do it in one place, we should avoid having all merchants implement it. Hoping this explains it better. ~Daniel |
Like this, good call. Merchant has less to think about regarding async / sync. |
agreed. a simple IF statement will fix it |
Adding an |
Good afternoon, a question does the sdk allow automatic charges? So there is a charge according to the plan or package that was previously sold to the user. |
Hi,
the charge method could also be used for convenience in case we see a soft decline (Order is suspenden => do not to a set call but do confirm and auth).
https://github.com/amzn/login-and-pay-with-amazon-sdk-php/blob/cbe6c69c0499d6b759b1dd468b302e3238068582/PayWithAmazon/Client.php#L1189
This will ease up the remedy workflow for merchants using this method.
Best wishes
Daniel
The text was updated successfully, but these errors were encountered: