fix(wallet): drop per-relay backup status list from Spark settings#572
Open
dmnyc wants to merge 1 commit into
Open
fix(wallet): drop per-relay backup status list from Spark settings#572dmnyc wants to merge 1 commit into
dmnyc wants to merge 1 commit into
Conversation
iOS doesn't surface a per-relay backup status display in the wallet settings screen, and the section is overkill for the Android side too — the majority of users start with the default Spark wallet (which never shows the relay-backup affordance at all, because the nsec is the canonical backup), and the remaining minority on non-default Spark wallets get the "Backup to Nostr Relays" button alongside the Recovery Phrase row without needing a per-relay status card to interpret. Removes the entire status block: the "Relay Backup Status" header + refresh icon, the relay-URL + green/grey dot card, the "Delete Relay Backup" TextButton + status messages, and the confirmation dialog. The "Backup to Nostr Relays" button stays — non-default wallets can still push the encrypted backup to relays, we just don't surface the fine-grained per-relay state inline. WalletViewModel plumbing (`relayBackupStatuses`, `relayBackupCheckLoading`, `checkRelayBackupStatuses()`, `deleteRelayBackup()`) is left in place deliberately. The status check + delete actions are still wired through the WalletScreen parameter list so other entry points (or a future debug-only view) can use them, and ripping the ViewModel state out would balloon this diff into a refactor.
This was referenced May 26, 2026
Open
Open
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
iOS doesn't surface a per-relay backup status display in the wallet settings, and the section is overkill on Android too:
What goes
IconButtonCardlisting each relay URL with a green / grey dotTextButton, status messages, and confirmationAlertDialogWhat stays
WalletViewModel.relayBackupStatuses/checkRelayBackupStatuses()/deleteRelayBackup()— kept deliberately in the ViewModel and on the screen's parameter list so other entry points (or a future debug-only surface) can still trigger them. Ripping the ViewModel state out would balloon this diff into a refactor.Files
app/src/main/kotlin/com/wisp/app/ui/screen/WalletScreen.kt(+7 / -131)Test plan