-
Notifications
You must be signed in to change notification settings - Fork 8
Dev 1806: Fix invalid schemas + add Sandbox Simulations endpoint for transfer processing #38
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
Conversation
| example: processor-sandbox-plaidauth-123456 | ||
| bankAccountType: | ||
| type: string | ||
| enum: [checking, savings, general-ledger, loan] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed these types because they aren't supported when adding banks via Plaid.
| example: https://api-sandbox.dwolla.com/on-demand-authorizations/30e7c028-0bdf-e511-80de-0aa34a9b2388 | ||
| bankAccountType: | ||
| type: string | ||
| enum: [checking, savings, general-ledger, loan] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same reason as above. Plaid Auth doesn't support Loan accounts.
| @@ -0,0 +1,58 @@ | |||
| post: | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Adding this endpoint so we can use it in the MCP and SDK.
In the developer-portal, however, if we'd like to hide it from the main reference we could do that using hidden pages.
| - Virtual Account: Creating Virtual Account Numbers (VANs) | ||
| oneOf: | ||
| - $ref: '#/CreateCustomerBankFundingSource' | ||
| - $ref: '#/CreateCustomerBankFundingSourceViaNumbers' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ShreyaThapa This is minor but is there a better name we could use instead of viaNumbers or ViaPlaid? It's already long, so I wonder if we should just name them:
CreateCustomerBankFundingSourceWithAccountNumbers
CreateCustomerBankFundingSourceWithPlaid
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Absolutely! I'll get that updated.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just pushed up a commit resolving this comment - 01a6879
No description provided.