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

provide bank account support #377

Open
wants to merge 2 commits into
base: original
Choose a base branch
from
Open

provide bank account support #377

wants to merge 2 commits into from

Conversation

lioman
Copy link

@lioman lioman commented Sep 27, 2017

  • add bank account model
  • adapt admin
  • sync source

What's this PR do?

Any background context you want to provide?

What ticket or issue # does this fix?

Closes #[issue number]

Definition of Done (check if considered and/or addressed):

  • Are all backwards incompatible changes documented in this PR?
  • Have all new dependencies been documented in this PR?
  • Has the appropriate documentation been updated (if applicable)?
  • Have you written tests to prove this change works (if applicable)?

@lioman
provide bank account suppport
8de9fc3 
- add bank account model
- adapt admin
- sync source
@paltman
Copy link
Member

paltman commented Sep 27, 2017

I think #336, which is very close to being merged, is going to take care of this already.

@blueyed
Copy link
Contributor

blueyed commented Oct 17, 2017

@paltman
BankAccount is different from Stripe Connect Accounts.

@blueyed
Copy link
Contributor

blueyed commented Oct 17, 2017

Should get rebased after #409 probably.

@blueyed
Copy link
Contributor

blueyed commented Oct 17, 2017

@lioman
you can rebase/update it before already, of course.. :)

@LukasKlement
Copy link

Are there any update on this pull request?

@lioman
Copy link
Author

lioman commented Jun 11, 2018

Sorry. We do not use it at the moment and I could not complete this. But it is the next thing on the list

@paltman paltman added the Original The original full pinax-stripe version with cached models and actions service layer label Nov 26, 2021
@paltman paltman changed the base branch from master to 4.x November 26, 2021 16:19
@paltman paltman changed the base branch from 4.x to original November 26, 2021 17:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Original The original full pinax-stripe version with cached models and actions service layer
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@lioman @paltman @blueyed @LukasKlement