Skip to content

[ONB-2513] Feat(v2): Add legal representative document upload#129

Merged
tlarrain merged 1 commit into
masterfrom
feat/onboarding-legal-representative-documents
Jul 7, 2026
Merged

[ONB-2513] Feat(v2): Add legal representative document upload#129
tlarrain merged 1 commit into
masterfrom
feat/onboarding-legal-representative-documents

Conversation

@tlarrain

@tlarrain tlarrain commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Description

Adds support for the new v2 legal representative document upload endpoint:

  • PUT /v2/entities/{entity_id}/onboardings/{onboarding_id}/legal_representatives/{legal_representative_id}/documents/{slot_key}

Also models the legal_representatives array that the API now returns on the onboarding object: new OnboardingLegalRepresentative resource, with its documents mapped to OnboardingDocument.

Requirements

None.

Additional changes

The onboarding create integration test now sends legal_representatives as an array, matching the API's current request shape (the singular legal_representative object no longer exists).

Adds upload_legal_representative_document to the onboardings manager
(PUT .../legal_representatives/{id}/documents/{slot_key}) and the
OnboardingLegalRepresentative resource, mapped from the onboarding's
new legal_representatives array. Onboarding create now takes
legal_representatives as an array, replacing the singular
legal_representative object.
@tlarrain tlarrain requested a review from JoMarchant July 7, 2026 13:28

@JoMarchant JoMarchant left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@tlarrain tlarrain merged commit 85db9ff into master Jul 7, 2026
5 checks passed
@tlarrain tlarrain deleted the feat/onboarding-legal-representative-documents branch July 7, 2026 14:49
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.

2 participants