Skip to content

Add Multisig announcement modal#281

Merged
mraveux merged 6 commits into
masterfrom
feature/multisig-announcement-modal
Nov 21, 2025
Merged

Add Multisig announcement modal#281
mraveux merged 6 commits into
masterfrom
feature/multisig-announcement-modal

Conversation

@onmax
Copy link
Copy Markdown
Member

@onmax onmax commented Oct 21, 2025

Summary

Adds a one-time announcement modal for Nimiq Multisig wallets.

Changes

  • Added MultisigAnnouncementModal component with mobile and desktop responsive design
  • Added multisig logo SVG and hero image assets
  • Integrated modal into app initialization flow (shows once per user)
  • Modal opens https://multisig.nimiq.com when button is clicked

TODO

  • Update blog post URL
  • Sync translations (i18n)
Screenshots

Desktop

image

Mobile

image

User Experience

New Users

New users will see this announcement modal once after completing their wallet setup, introducing them to the Multisig feature.

Current Users

Existing users will see this modal once on their next visit to the wallet, as the "shown once" state is tracked per browser via localStorage.

Implementation

  • Follows existing modal patterns (PolygonActivationModal, BtcActivationModal)
  • Uses localStorage to track "shown once" state
  • Shows after USDC activation check, before WelcomeStaking modal
  • Clean, minimal code with comments only for "why", not "what"

Testing

Run dev servers for wallet, hub, and keyguard to test the modal flow.

@onmax onmax force-pushed the feature/multisig-announcement-modal branch from 585b152 to 5f4b7fc Compare October 22, 2025 13:18
@onmax onmax marked this pull request as ready for review October 22, 2025 13:19
@onmax onmax force-pushed the feature/multisig-announcement-modal branch from 5f4b7fc to 19e3518 Compare October 22, 2025 13:20
@mraveux mraveux self-assigned this Oct 28, 2025
mraveux added a commit that referenced this pull request Oct 28, 2025
mraveux added a commit that referenced this pull request Oct 28, 2025
mraveux added a commit that referenced this pull request Oct 28, 2025
Copy link
Copy Markdown
Member

@mraveux mraveux left a comment

Choose a reason for hiding this comment

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

I pushed some commits but now I think it's good to go, if you're ok with the changes. 👍
Translations have also been synced and we're now only missing the blog URL and https://multisig.nimiq.com/ to be online (since it seems to be unreachable for now).
I will approve the PR but please only merge it once those 2 things are taken care of.
And let me know if / when you need a deployment, I can also do it for you. 👍

@onmax
Copy link
Copy Markdown
Member Author

onmax commented Oct 29, 2025

Please feel free to merge and deploy it whenever it is ready.

Thanks!

mraveux added a commit that referenced this pull request Nov 20, 2025
mraveux added a commit that referenced this pull request Nov 20, 2025
mraveux added a commit that referenced this pull request Nov 20, 2025
@mraveux mraveux force-pushed the feature/multisig-announcement-modal branch 2 times, most recently from 4887935 to 9321197 Compare November 20, 2025 00:29
@mraveux mraveux force-pushed the feature/multisig-announcement-modal branch from 9321197 to 13614a4 Compare November 21, 2025 17:02
@mraveux mraveux merged commit bff4144 into master Nov 21, 2025
1 check passed
@mraveux mraveux deleted the feature/multisig-announcement-modal branch November 21, 2025 17:18
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