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

Braintree: Add risk data fields to response #3169

Merged
merged 1 commit into from
Mar 12, 2019

Conversation

jknipp
Copy link
Member

@jknipp jknipp commented Mar 12, 2019

Explicitly add the risk data fields to the active merchant response, if
they are provided.

ECS-210

Unit:
59 tests, 158 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed

Remote:
71 tests, 407 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed

@jknipp jknipp self-assigned this Mar 12, 2019
@jknipp jknipp requested review from deedeelavinder and a team March 12, 2019 15:42
Copy link
Contributor

@bayprogrammer bayprogrammer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@jknipp jknipp force-pushed the ECS-210-braintree-risk-data branch from 53f55b0 to a7b4d38 Compare March 12, 2019 21:32
Explicitly add the risk data fields to the active merchant response, if
they are provided.

ECS-210

Unit:
59 tests, 158 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed

Remote:
71 tests, 407 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed

closes activemerchant#3169
@jknipp jknipp force-pushed the ECS-210-braintree-risk-data branch from a7b4d38 to 0629575 Compare March 12, 2019 21:37
@jknipp jknipp merged commit 0629575 into activemerchant:master Mar 12, 2019
@jknipp jknipp deleted the ECS-210-braintree-risk-data branch March 12, 2019 21:42
whitby3001 pushed a commit to whitby3001/active_merchant that referenced this pull request Sep 3, 2019
Explicitly add the risk data fields to the active merchant response, if
they are provided.

ECS-210

Unit:
59 tests, 158 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed

Remote:
71 tests, 407 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications
100% passed

closes activemerchant#3169
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants