Skip to content

Add disabledWallets prop, hide UniSat by default#25

Open
BlobMaster41 wants to merge 1 commit into
masterfrom
unisat-disabled-by-default
Open

Add disabledWallets prop, hide UniSat by default#25
BlobMaster41 wants to merge 1 commit into
masterfrom
unisat-disabled-by-default

Conversation

@BlobMaster41
Copy link
Copy Markdown
Collaborator

Summary

UniSat does not support MLDSA signatures or full OPNet transaction signing. Instead of removing it entirely, this adds a configurable disabledWallets prop to WalletConnectProvider.

Default behavior: UniSat is hidden from the connect modal.
Opt-in: Pass disabledWallets={[]} to show all registered wallets.

API

// Default: UniSat hidden
<WalletConnectProvider theme="dark">

// Explicit: show all wallets including UniSat
<WalletConnectProvider theme="dark" disabledWallets={[]}>

// Custom: hide specific wallets
<WalletConnectProvider theme="dark" disabledWallets={[SupportedWallets.UNISAT]}>

Changes

Single file change in WalletConnectProvider.tsx:

  • Added disabledWallets prop (defaults to [SupportedWallets.UNISAT])
  • Filters allWallets, availableWallets, and the fallback list through the disabled filter
  • All UniSat code remains intact for devs who want it

Supersedes #24 based on community feedback.

UniSat does not support MLDSA signatures or full OPNet transaction signing.
Rather than removing it entirely, this adds a `disabledWallets` prop to
WalletConnectProvider that defaults to `[SupportedWallets.UNISAT]`.

Developers who want UniSat can opt in with `disabledWallets={[]}`.
Developers who want to hide other wallets can pass a custom array.

Default behavior: UniSat is hidden from the modal.
Opt-in behavior: pass disabledWallets={[]} to show all wallets.
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