Skip to content

[ONB-2512] Feat(v2): Add legal representative document upload#102

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

[ONB-2512] Feat(v2): Add legal representative document upload#102
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 adds the OnboardingLegalRepresentative resource class for the legal_representatives array that the API now returns on the onboarding object, mirroring how OnboardingShareholder is modeled.

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).

uploadLegalRepresentativeDocument PUTs the file to /v2/entities/{entity_id}/onboardings/{id}/legal_representatives/{legal_representative_id}/documents/{slot_key}, mirroring the shareholder document upload. Adds the OnboardingLegalRepresentative resource for the legal_representatives array now present in onboarding responses, and updates the create() example to the new legal_representatives array form (the singular legal_representative object is gone from the API).
@tlarrain tlarrain requested a review from JoMarchant July 7, 2026 13:29

@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 9de267f into master Jul 7, 2026
2 checks passed
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