audius-protocol icon indicating copy to clipboard operation
audius-protocol copied to clipboard

WIP Adopt Reown for Connected Wallets, remove web3.js

Open rickyrombo opened this issue 11 months ago • 1 comments

This one became a doozy:

  • Moves our "Web3Modal" to Reown AppKit (much prettier, more wallet support, allows removal of web3.js!)
  • Moves the ConnectedWallets modal out of the overloaded WalletModal
  • Migrates Connected Wallets to use TanStack-Query instead of the convoluted sagas
    • Including new queries for fetching collectibles and balances for a wallet

TODO:

  • Fix mobile
  • Fix mobile web
  • Fix interaction with "Sign In With Metamask" - using the same wagmi provider for both is sure to cause problems, and will probably end up disconnecting the user once we update the "Sign in/up with metamask" to Reown as well if it doesn't already

rickyrombo avatar Feb 21 '25 09:02 rickyrombo

⚠️ No Changeset found

Latest commit: 90b5dad0b707761d426b551cc6dfeeec6f04e3f9

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

changeset-bot[bot] avatar Feb 21 '25 17:02 changeset-bot[bot]