-
Notifications
You must be signed in to change notification settings - Fork 91
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
Description Option not being sent #43
Comments
I'm trying this out in my test account, and I can't see the description getting through to the gateway at all - the transactions show up on the transaction search control panel as all empty. Any idea what may be happening here? The PR is also failing tests, and I'm not sure why. I suspect it may be the blank lines full of spaces that your editor puts in instead of just blank lines. |
I can't get this PR to work, can't get the tests to pass locally (whether the tests are wrong or the driver is wrong, I really don't know). Travis tests are also failing. Personally can't afford to spend any more time on this PR. |
@judgej I added a new pr that modifies bd86's code slightly. Unit tests pass and I am seeing the description go through to Authorize.net |
This has been merged now. I added the invoiceNumber at the same time, as it made sense. I couldn't find an invoice number field in OmniPay Common, though I could have sworn I remember there being one there. Probably just tired. |
In release 2.5.0 |
The description transaction option was not being set in AIMGateway. Created pull request #44
The text was updated successfully, but these errors were encountered: