-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Realex: Remove support for network tokenization #3638
base: master
Are you sure you want to change the base?
Realex: Remove support for network tokenization #3638
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👀 What is that test output? I get one unrelated local remote test failing, which is also failing for me on master: |
@molbrown what is the error you are getting? It is passing for me locally:
|
Sorry I missed this, I get a different result code in |
ECS-1186 Discussions with the Realex support team have revealed that these fields are not actually supported. This pull request removes support for network tokenized cards on the Realex gateway. rake test:local 4503 tests, 71981 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications 100% passed test/remote/gateways/remote_realex_test.rb 25 tests, 131 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications 100% passed
cfcee5e
to
d8eafcf
Compare
I updated my commit message with local and remote test outputs 👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Thanks, looks great.
Why?
Discussions with the Realex support team have revealed that these fields are not actually supported.
What Changed?
This pull request removes support for network tokenized cards on the Realex gateway.
Test Suite