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

Description Option not being sent #43

Closed
bd86 opened this issue Jun 29, 2016 · 5 comments
Closed

Description Option not being sent #43

bd86 opened this issue Jun 29, 2016 · 5 comments

Comments

@bd86
Copy link

bd86 commented Jun 29, 2016

The description transaction option was not being set in AIMGateway. Created pull request #44

@judgej
Copy link
Member

judgej commented Jul 7, 2016

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.

@judgej
Copy link
Member

judgej commented Jul 16, 2016

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.

@balexander
Copy link
Contributor

@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

@judgej
Copy link
Member

judgej commented Nov 10, 2016

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.

@judgej
Copy link
Member

judgej commented Mar 7, 2017

In release 2.5.0

@judgej judgej closed this as completed Mar 7, 2017
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

3 participants