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

Latest AVS Rules support PR breaks transactions without address #65

Open
jason-nabooki opened this issue Apr 29, 2021 · 0 comments
Open

Comments

@jason-nabooki
Copy link

When PR #36 was merged in, it broke transactions that do not contain an address. Braintree doesn't require an address for a transaction but it does if you have AVS rules configured.

By default AVS rules are not configured so this can be replicated with a new Braintree sandbox account.

If you try to make a transaction without an address the Braintree API will reject it with the following error...

message=Addresses must have at least one field filled in

For more information on this error see https://developers.braintreepayments.com/reference/general/validation-errors/all/ruby#code-81801

@barrycarrjr

ivanmitrikeski added a commit to AutoprintSoftware/omnipay-braintree that referenced this issue May 11, 2021
barryvdh pushed a commit that referenced this issue May 11, 2021
…dress #65 (#66)

* AVS Rules Updates

* Test updates

* Fix Tests for CreatePaymentMethodRequest
added doc blocks
used array merge instead of +=

* Issue #65 Fix

Co-authored-by: Ivan Mitrikeski <imitrikeski@rcs.us>
Co-authored-by: Ivan Mitrikeski <ivan@mitrik.com>
Co-authored-by: Ivan Mitrikeski <imitrik@gmail.com>
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

No branches or pull requests

1 participant