Skip to content

fix(sdk-core): forward videoApprovers on TSS txrequests - #9966

Merged
Charan-Ratlan merged 1 commit into
masterfrom
WCI-1744
Oct 15, 2026
Merged

Charan-Ratlan merged 1 commit into
masterfrom
WCI-1744

Conversation

@Charan-Ratlan

Copy link
Copy Markdown
Contributor

The Video ID approvers selected for a TSS withdrawal were dropped before the /txrequests call, so Wallet Platform stored videoApprovers: [] and emailed the enterprise primary contact instead.

Add an optional videoApprovers option and send it as a top-level field of the /txrequests body for transfer and transfertoken. It is omitted from the request when not provided.

Ticket: WCI-1744

The Video ID approvers selected for a TSS withdrawal were dropped before
the /txrequests call, so Wallet Platform stored videoApprovers: [] and
emailed the enterprise primary contact instead.

Add an optional videoApprovers option and send it as a top-level field
of the /txrequests body for transfer and transfertoken. It is omitted
from the request when not provided.

Ticket: WCI-1744
@linear-code

linear-code Bot commented Oct 14, 2026

Copy link
Copy Markdown
Contributor

WCI-1744

@github-actions

Copy link
Copy Markdown
Contributor

⚠️ Unit tests are failing on Node 26.x (Current release line, non-blocking). This is not an LTS version yet, so it does not block merge, but it signals an incompatibility to fix before Node 26.x becomes LTS.

View run

@Charan-Ratlan
Charan-Ratlan marked this pull request as ready for review October 14, 2026 13:31
@Charan-Ratlan
Charan-Ratlan requested review from a team as code owners October 14, 2026 13:31
@Charan-Ratlan
Charan-Ratlan merged commit 68101aa into master Oct 15, 2026
26 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