-
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
CyberSource: Update Stored Credential flow #5136
CyberSource: Update Stored Credential flow #5136
Conversation
@@ -862,11 +866,21 @@ def add_threeds_2_ucaf_data(xml, payment_method, options) | |||
def stored_credential_commerce_indicator(options) | |||
return unless options[:stored_credential] | |||
|
|||
return if options[:stored_credential][:initial_transaction] | |||
if options[:stored_credential][:initial_transaction] |
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 think it's best just to remove lines 869-875 since it doesn't work and we haven't heard back from Cybersource.
2eb3f92
to
edcfdde
Compare
0dfe063
to
79c1c43
Compare
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.
Looks good.
I do see something in the ticket about recurring_internet
as a commerce_indicator, but I don't see it in the code. Is that something that we need to support?
I ended up removing that because it wasn't working when testing remotely and I didn't hear back from the CyberSource on why this was failing. |
Always send commerceIndicator if stored credential reason_type is present. Update the value send for NetworkTokens commerceIndicator based on if they are using stored credentials. Spreedly reference: [ECS-3532](https://spreedly.atlassian.net/browse/ECS-3532) Unit tests Finished in 26.745895 seconds. 5912 tests, 79756 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications 100% passed RuboCop 798 files inspected, no offenses detected
79c1c43
to
c5a4d22
Compare
Spreedly reference:
ECS-3532
Unit tests
Finished in 26.745895 seconds.
5912 tests, 79756 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications 100% passed
RuboCop
798 files inspected, no offenses detected