Skip to content

feat: "Generate new address" button to wallet address list views - #1358

Draft
sneurlax wants to merge 2 commits into
stagingfrom
feat/430-generate-address-button
Draft

feat: "Generate new address" button to wallet address list views#1358
sneurlax wants to merge 2 commits into
stagingfrom
feat/430-generate-address-button

Conversation

@sneurlax

Copy link
Copy Markdown
Member

closes #430

@codecov

codecov Bot commented May 28, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 94 lines in your changes missing coverage. Please review.
✅ Project coverage is 4.83%. Comparing base (1139770) to head (6af5dc8).
⚠️ Report is 2130 commits behind head on staging.

Files with missing lines Patch % Lines
.../receive_view/addresses/wallet_addresses_view.dart 0.00% 61 Missing ⚠️
...cific/addresses/desktop_wallet_addresses_view.dart 0.00% 33 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           staging    #1358      +/-   ##
===========================================
- Coverage     5.36%    4.83%   -0.53%     
===========================================
  Files          797     1092     +295     
  Lines        75915   109196   +33281     
===========================================
+ Hits          4075     5283    +1208     
- Misses       71840   103913   +32073     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@sneurlax

Copy link
Copy Markdown
Member Author

Converting to draft because I have too many open PRs ready--will take another review pass over this before re-readying it

@sneurlax
sneurlax marked this pull request as draft August 20, 2026 19:00
No behaviour change; the formatter is enforced on changed files in CI.
Both the mobile and desktop address lists gain a guarded button that
derives the next receiving address. GenerateAddressButton owns the whole
flow: it suppresses repeat taps, shows a non dismissible progress dialog
on the root navigator, closes exactly that dialog's route when
generation settles, and surfaces failures instead of stranding the
modal. The button is only offered to wallets that can actually derive
another address, so view only wallets holding just an address or a spark
view key do not get it.

closes #430
@sneurlax
sneurlax force-pushed the feat/430-generate-address-button branch from 1f8f492 to da800a2 Compare August 24, 2026 21:31
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.

Add generate new address button in address list

1 participant