Skip to content

Conversation

@ShreyaThapa
Copy link
Contributor

  • Updates GET /customers/{id}/funding-sources 200 response to remove redundant "type: array". this fixes Issue #21 in dwolla-typescript
  • Updated GET /business-classifications 200 response to use content.application/vnd.dwolla.v1.hal+json.schema and reference #/components/schemas/BusinessClassifications (fixes invalid title/type/properties placement).
  • Added components.schemas.BusinessClassifications with HAL structure: _links, _embedded.business-classifications[], and total.
  • Added country to components.schemas.Address.properties and kept it in required.
  • Removed format: date from several dateOfBirth fields to match actual API payloads
  • Add new schemas for handling business verified retry scenarios
    • business only retry,
    • business+ controller retry,
    • business + International controller retry

@ShreyaThapa ShreyaThapa changed the title Fixes issue 21 on dwolla-typescript repo Fixes issue 21 on dwolla-typescript repo + Additional updates Oct 1, 2025
required:
- address1
- city
- country
Copy link
Member

@spencerhunter spencerhunter Oct 1, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is postalCode not a required param?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Correct, it's not required for International Controllers address.

@ShreyaThapa ShreyaThapa merged commit 7138084 into main Oct 1, 2025
5 checks passed
@ShreyaThapa ShreyaThapa deleted the fix/ts-issue-21 branch October 1, 2025 15:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants