rainbowkit icon indicating copy to clipboard operation
rainbowkit copied to clipboard

feat: add TokenPocket wallet

Open chendatony31 opened this issue 2 years ago • 2 comments

chendatony31 avatar Jun 28 '22 16:06 chendatony31

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
rainbowkit-example ✅ Ready (Inspect) Visit Preview Oct 9, 2022 at 8:22PM (UTC)
rainbowkit-site ✅ Ready (Inspect) Visit Preview Oct 9, 2022 at 8:22PM (UTC)

vercel[bot] avatar Jun 28 '22 16:06 vercel[bot]

Thanks for the contribution here @chendatony31 – we're currently discussing out strategy for adding new wallets to the repo and how we can best serve the RainbowKit community while also not being gatekeepers/maintainers of every wallet integration. I'll let you know when we have an update! Thanks for your patience!

nickbytes avatar Jul 19 '22 22:07 nickbytes

@markdalgleish is attempting to deploy a commit to the rainbowdotme Team on Vercel.

A member of the Team first needs to authorize it.

vercel[bot] avatar Sep 30 '22 00:09 vercel[bot]

Sorry for the delay on this, and thanks for the PR!

We've just re-worked RainbowKit internals in v0.7 so that wallets can be imported individually, ensuring our bundle size doesn't keep increasing as we continue to add more wallets.

Thanks for giving us access to your fork — I've pushed some updates to your PR migrating it to the new API. Once I've prepared a few more wallet PRs to batch together, I'll get this one merged.

markdalgleish avatar Sep 30 '22 04:09 markdalgleish

I just tried this integration and noticed the following:

  • On Android, tapping the TokenPocket icon opens the app but then nothing happens. At this point the app regularly crashes on me too. Have you been able to get this to work?
  • On iOS, after connecting I'm not prompted to go back to my browser (or redirected automatically). Not a bug, but just flagging that there's a bit of friction there.
  • On iOS, I'm not taken to the app store if the app isn't installed. Instead, I get an alert saying "Safari cannot open the page because the address is invalid." Again, not necessarily a bug, but I'd recommend looking into universal links as other iOS wallets are using in RainbowKit.
  • On desktop, I'm not presented with a WalletConnect QR code even though the mobile apps support this. If I select WalletConnect on desktop instead of TokenPocket, I am able to connect via QR code. I recommend looking at the other WalletConnect-based wallets to get an idea of how this works. We also support presenting users with download links for both the browser extension and the mobile app to ensure that they can still get the browser extension even though they're presented with the QR code.

markdalgleish avatar Oct 04 '22 01:10 markdalgleish

@markdalgleish Thank you very much for these good suggestions, we have fix these questions except the second one, which need some time for our wallet to fix it.

chendatony31 avatar Oct 08 '22 12:10 chendatony31

@chendatony31 Circling back on this: are the issues resolved? Would be happy to look at reviewing again

DanielSinclair avatar Jan 19 '23 00:01 DanielSinclair