Skip to content

fix(pending-approval): cancel pending approvals - #9748

Draft
ralph-bitgo[bot] wants to merge 1 commit into
masterfrom
WCI-1631-cancel-pending-transactions
Draft

ralph-bitgo[bot] wants to merge 1 commit into
masterfrom
WCI-1631-cancel-pending-transactions

Conversation

@ralph-bitgo

@ralph-bitgo ralph-bitgo Bot commented Sep 16, 2026

Copy link
Copy Markdown

What

  • Add an explicit canceled pending-approval state to the SDK model and V2 response contract.
  • Route V1 and V2 cancellation requests to PendingApproval.cancel().
  • Make both SDK implementations send state: canceled instead of aliasing cancellation to rejection.
  • Add SDK and Express endpoint coverage for cancellation.

Why

Pending transactions need a distinct cancellation path so callers can cancel them without incorrectly reporting them as rejected. Rejection behavior remains unchanged for all other non-approval states.

Test plan

  • nix develop . --command yarn lerna run build --stream
  • Focused core cancellation test: 1 passing
  • Focused Express cancellation tests: 2 passing
  • Targeted ESLint and Prettier checks: passing

Ticket: WCI-1631

Route cancellation requests to the pending approval cancel API and send the canceled state from both SDK implementations. Extend the V2 response contract and add endpoint and SDK coverage.

This preserves rejection semantics while allowing pending transactions to be canceled explicitly instead of being reported as rejected.

Ticket: WCI-1631
Session-Id: 8aad1ca5-334f-4dee-a4e7-bd8cd69a92e3
Task-Id: c637ca13-8d29-4521-b341-6bf9a94fbc44
@ralph-bitgo
ralph-bitgo Bot force-pushed the WCI-1631-cancel-pending-transactions branch from 7b8c37b to c3d88f5 Compare September 16, 2026 09:36
@github-actions

github-actions Bot commented Sep 16, 2026

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

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.

0 participants