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

Added currency to Transaction refund method #40

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

silasmontgomery
Copy link

  1. Added $currency param to method
  2. Added withCurrency to the builder

Prior to this change was receiving error 'currency cannot be null for this transaction type' (heartland/portico).

1. Added $currency param to method
2. Added withCurrency to the builder
@silasmontgomery
Copy link
Author

Well, I definitely didn't check out the previous pull requests (#34) prior to making this one. This method of refunding is shown on "Full SDK Documentation > Credit Card" on Heartlands developer site (https://developer.heartlandpaymentsystems.com/Documentation/credit-card-payments) and is useful because you can refund from a transaction ID only. When using tokenization (not saving card data) I'm unsure of how to use $card->refund.

@hpsTony
Copy link

hpsTony commented Nov 9, 2019 via email

@silasmontgomery
Copy link
Author

silasmontgomery commented Nov 11, 2019

Thanks, that will work when requesting/saving a multi-use token but in the case of one-time charges it would seem the transaction ID would be the only way to refund without having the actual card data, correct?

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

Successfully merging this pull request may close these issues.

2 participants