Skip to content

feat: multisig UX improvements#560

Merged
n13 merged 7 commits into
mainfrom
feat/multisig-ux-improvements
Jul 10, 2026
Merged

feat: multisig UX improvements#560
n13 merged 7 commits into
mainfrom
feat/multisig-ux-improvements

Conversation

@n13

@n13 n13 commented Jul 10, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Fix fee calculation to show proposer's balance (not multisig balance) and display specific "Insufficient Balance for Fee" error
  • Replace "TO" row with "PROPOSER" showing full address + checkphrase; remove all abbreviated addresses from review screen
  • Client-side fallback decoding of proposal amount/recipient from call_raw when indexer fails
  • Change "APPROVED" pill to "SIGNED" to avoid confusion about overall proposal status
  • Add tilde prefix to estimated expiry time; enforce minimum 24h expiry
  • Silent refresh for open proposals (no visible loading swirl)
  • Add "DONE" button to dismiss proposal detail sheet after signing

Test plan

  • Create a multisig proposal and verify proposer balance and fee error display correctly
  • Review screen shows full addresses and proposer row
  • Existing proposal with indexer decode failure still shows amount and recipient
  • Status pills show "SIGNED" not "APPROVED"
  • Open proposals refresh silently without loading indicator
  • After signing, "Done" button appears to dismiss the sheet

Comment thread mobile-app/lib/v2/screens/send/input_amount_screen.dart Outdated

@dewabisma dewabisma left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM just small nit.

@n13 n13 merged commit 2bd2095 into main Jul 10, 2026
1 check 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