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

Add CustomerRequest #212

Merged
merged 1 commit into from Feb 28, 2018
Merged

Add CustomerRequest #212

merged 1 commit into from Feb 28, 2018

Conversation

leighmcculloch
Copy link
Contributor

What

Add CustomerRequest and change the CustomerGateway Create function to use
the same patterns found in the official Braintree SDKs.

Why

Not all the fields that are returned for a customer are valid to pass to
Braintree when creating and updating customers. The official Braintree
SDKs, and some of the other gateways in this SDK, have moved to the
model of separating requests and responses. This is part of the
continuing effort to bring this SDK inline with the official SDKs.

Notes

This contains a breaking change. The object used to hold the parameters
for the CustomerGateway's Create function has changed from Customer to
CustomerRequest. This is to be consistent with the other SDKs and so
that the request struct only contains fields that can be passed to
Braintree which makes its use clearer.

Related

#209

What
===
Add CustomerRequest and change the CustomerGateway Create function to use
the same patterns found in the official Braintree SDKs.

Why
===
Not all the fields that are returned for a customer are valid to pass to
Braintree when creating and updating customers. The official Braintree
SDKs, and some of the other gateways in this SDK, have moved to the
model of separating requests and responses. This is part of the
continuing effort to bring this SDK inline with the official SDKs.

Notes
===
This contains a breaking change. The object used to hold the parameters
for the CustomerGateway's Create function has changed from Customer to
CustomerRequest. This is to be consistent with the other SDKs and so
that the request struct only contains fields that can be passed to
Braintree which makes its use clearer.
@leighmcculloch leighmcculloch merged commit 616110a into master Feb 28, 2018
@leighmcculloch leighmcculloch deleted the customer-request branch February 28, 2018 06:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant