Skip to content

Show performance warning when enabling Nym on multiple assets#5944

Merged
j0ntz merged 1 commit intodevelopfrom
jon/nym-multi-asset-warning
Feb 20, 2026
Merged

Show performance warning when enabling Nym on multiple assets#5944
j0ntz merged 1 commit intodevelopfrom
jon/nym-multi-asset-warning

Conversation

@j0ntz
Copy link
Contributor

@j0ntz j0ntz commented Feb 17, 2026

CHANGELOG

Does this branch warrant an entry to the CHANGELOG?

  • Yes

Dependencies

None

Description

Context

Asana: "Show warning when enabling more than 1 asset with Nym" (P1). From the 2/10 huddle: show a warning when enabling Nym on multiple assets, but don't lock functionality.

Changes

  • Added a ConfirmContinueModal warning that appears when a user enables Nym Mixnet on a second (or more) asset. The warning explains that multi-asset Nym usage may slow down sends and delay balance updates.
  • The acknowledgment is persisted to LocalAccountSettings (isNymWarningShown) so the modal only appears once per account, matching the pattern used by token gas fee warnings.
  • The warning is shown in both the dedicated Privacy Settings scene (switch toggles) and the individual currency settings page (radio buttons).

Requirements

If you have made any visual changes to the GUI. Make sure you have:

  • Tested on iOS device
  • Tested on Android device
  • Tested on small-screen device (iPod Touch)
  • Tested on large-screen device (tablet)

@j0ntz j0ntz force-pushed the jon/nym-multi-asset-warning branch 3 times, most recently from 144c3ad to c195071 Compare February 20, 2026 00:24
When a user enables Nym Mixnet on a second (or more) asset, show a
ConfirmContinueModal warning that multi-asset Nym may slow sends and
delay balance updates. The acknowledgment is persisted to local account
settings so the modal only appears once per account.

The warning is shown in both the dedicated Privacy Settings scene and
the individual currency settings radio buttons.
@j0ntz j0ntz force-pushed the jon/nym-multi-asset-warning branch 3 times, most recently from f298028 to 5f1f5a7 Compare February 20, 2026 00:37
@j0ntz j0ntz merged commit 409046b into develop Feb 20, 2026
4 checks passed
@j0ntz j0ntz deleted the jon/nym-multi-asset-warning branch February 20, 2026 00:38
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.

3 participants

Comments