Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What's this PR do?
This PR adds the ability to remove an external bank account from account.
Any background context you want to provide?
Right now, we are able to sync accounts (update/add) from current stripe data, but it would be great to have actions to do with deletions.
For example: User creates a custom account, which automatically creates an external bank account.
If this user changes their payment method and decides to add new account (using stripe API), we can use
sync_account()
to update our account/external_bank_accounts.However, the user now wants to delete their previous (very first external bank account). We do this using stripe API:
This time, we need to manually remove this from
BankAccount
object, because syncing the data will not remove, but instead only add/update existing accounts.What ticket or issue # does this fix?
Closes #569
Definition of Done (check if considered and/or addressed):