Add city to AddCustomer and AddShipping, allow for updating payment profile billing address. #41
Conversation
|
Note that it might be better to add an overload that accepts city so that this change won't be a breaking change. Thoughts? |
|
I added this to #46 |
It appears that the only way to get a billing address on a customer payment profile is on the Add methods (add e-check or add credit card). I can't see any direct way to update the billing address after the fact. This commit tries to solve this.
drmohundro
changed the title
|
I added another small change to allow for updating the billing address when calling I'd be happy to squash these commits for a cleaner merge if that helps. |
|
Yes, even better, consolidate all of these conflicting commits into one. Makes it more likely they will get merged. |
|
Agreed - do you think it makes more sense to put it all in #46 given that you've got more commits? |
|
No problem, I can do that. |
|
that should do it for this PR. Hopefully someone will merge in the PR soon. |
|
We're going to take these two PRs from the combined #46 Thanks guys! Brian |
This would fix #40.