Skip to content

fix: rpc send tranfer insufficient funds error handling - #2526

Draft
edgar-stacks wants to merge 3 commits into
devfrom
fix/rpc-send-transfer-insufficient-funds
Draft

edgar-stacks wants to merge 3 commits into
devfrom
fix/rpc-send-transfer-insufficient-funds

Conversation

@edgar-stacks

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions

github-actions Bot commented Jul 17, 2026 •

Copy link
Copy Markdown
Contributor

Leather Web build fed1d51 — Web staging

@github-actions

github-actions Bot commented Jul 17, 2026 •

Copy link
Copy Markdown
Contributor

Leather Extension build fed1d51 — Extension build

@codecov

codecov Bot commented Jul 17, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 32.08556% with 127 lines in your changes missing coverage. Please review.
✅ Project coverage is 16.84%. Comparing base (a4d4af9) to head (06636da).
⚠️ Report is 1 commits behind head on dev.

Files with missing lines Patch % Lines
.../app/pages/rpc-send-transfer/rpc-send-transfer.tsx 0.00% 64 Missing and 1 partial ⚠️
...logs/switch-account-sheet/switch-account-sheet.tsx 0.00% 13 Missing ⚠️
...pc-send-transfer/use-rpc-send-transfer-actions.tsx 0.00% 12 Missing ⚠️
...sion/src/app/store/keychains/keychain.selectors.ts 14.28% 12 Missing ⚠️
...uest/signing-account-card/signing-account-card.tsx 76.19% 5 Missing ⚠️
...switch-account/use-switch-account-sheet-context.ts 0.00% 4 Missing ⚠️
...extension/src/app/features/container/container.tsx 0.00% 4 Missing ⚠️
...fee-editor/bitcoin/bitcoin-fee-editor.provider.tsx 0.00% 4 Missing ⚠️
.../rpc-send-transfer/rpc-send-transfer-container.tsx 0.00% 3 Missing and 1 partial ⚠️
...rc/app/features/fee-editor/fee-editor.provider.tsx 0.00% 2 Missing ⚠️
... and 2 more
Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #2526      +/-   ##
==========================================
+ Coverage   16.77%   16.84%   +0.07%     
==========================================
  Files        2535     2535              
  Lines      110794   110941     +147     
  Branches     5624     5639      +15     
==========================================
+ Hits        18582    18690     +108     
- Misses      91226    91267      +41     
+ Partials      986      984       -2     
Files with missing lines Coverage Δ
...switch-account-sheet/switch-account-sheet.utils.ts 100.00% <100.00%> (ø)
...eatures/fee-editor/bitcoin/bitcoin-fees-loader.tsx 76.19% <100.00%> (+76.19%) ⬆️
.../src/app/features/fee-editor/fee-editor.context.ts 0.00% <ø> (ø)
...on/src/app/common/switch-account/switch-account.ts 0.00% <0.00%> (ø)
...ps/web/app/pages/multisig/account/account.page.tsx 0.00% <0.00%> (ø)
...rc/app/features/fee-editor/fee-editor.provider.tsx 0.00% <0.00%> (ø)
...switch-account/use-switch-account-sheet-context.ts 0.00% <0.00%> (ø)
...extension/src/app/features/container/container.tsx 0.00% <0.00%> (ø)
...fee-editor/bitcoin/bitcoin-fee-editor.provider.tsx 0.00% <0.00%> (ø)
.../rpc-send-transfer/rpc-send-transfer-container.tsx 0.00% <0.00%> (ø)
... and 5 more

... and 1 file with indirect coverage changes

Components Coverage Δ
bitcoin 82.88% <ø> (ø)
query 26.74% <ø> (ø)
utils 89.04% <ø> (ø)
crypto 78.20% <ø> (ø)
stacks 62.64% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

This branch was previously deployed

1 inactive deployment
web:staging — fed1d517 Deployed Jul 22, 2026 by edgar-stacks via staging #615
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.

handle INSUFFICIENT_FUNDS error when dApp triggers sendTransfer with amount higher than max send amount

1 participant