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.

Sign up for GitHub

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

Jump to bottom

BankAccounts should not take an address #686

Open
msherry opened this issue Aug 25, 2014 · 1 comment · May be fixed by #706
Open

BankAccounts should not take an address #686

msherry opened this issue Aug 25, 2014 · 1 comment · May be fixed by #706
Milestone
Revision 1.2

Comments

@msherry
Copy link
Contributor

msherry commented Aug 25, 2014

It makes sense for some funding instrument types to have an associated address -- credit/debit cards have a billing address, checks would have a physical address to which we could mail them, etc. Bank accounts have no concept of an address, and shouldn't allow them. This was added in 1.1 -- I suspect it was accidental.
@remear remear linked a pull request Oct 6, 2014 that will close this issue
Open
@rserna2010
Copy link
Contributor

Bank account addresses are returned in the view. Removal from the API needs to be done in the next API version as it is a breaking change.

@mjallday mjallday added this to the Revision 1.2 milestone Oct 7, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
Revision 1.2
Development

Successfully merging a pull request may close this issue.

Remove address from the BankAccount resource balanced/balanced-api
3 participants
@msherry @mjallday @rserna2010