-
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
Redsys: 3ds support #3336
Redsys: 3ds support #3336
Conversation
ddb6305
to
985e95f
Compare
data[:order_id].to_s + | ||
@options[:login].to_s + | ||
data[:currency] | ||
data[:order_id].to_s + |
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.
Was this white space intended?
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.
I added it after getting a rubocop error, i'll double check 👍
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.
🐘
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.
985e95f
to
26e5261
Compare
This PR preps Redsys for 3DS integration. It adds the test and production URLs for the 3DS service, updates the commit method to interact with the appropriate endpoint for 3DS transactions, and updates the parse method to appropriately parse responses. Unit: 34 tests, 104 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications 100% passed Remote: 20 tests, 63 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications 100% passed
26e5261
to
74506fc
Compare
This PR adds 3DS support to Redsys. It adds the test and production
URLs for the 3DS service, updates the commit method to interact with the
appropriate endpoint for 3DS transactions, and updates the parse method
to appropriately parse responses.
Unit:
34 tests, 104 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed
Remote:
20 tests, 63 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed