-
Notifications
You must be signed in to change notification settings - Fork 642
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(CoinbaseWalletOptions): add preference #2002
feat(CoinbaseWalletOptions): add preference #2002
Conversation
@wilsoncusack is attempting to deploy a commit to the rainbowdotme Team on Vercel. A member of the Team first needs to authorize it. |
Thanks @wilsoncusack. Looking into this now 👍 |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@wilsoncusack Changes looks great. I have a quick question. When setting Works great with |
@wilsoncusack Gotcha. Alright we'll have a last round of review and cut a release early next week. Thanks for getting this done👍 |
@wilsoncusack Closing this in favor of #2017 |
The Coinbase Wallet Wagmi connector now has a
preference
argument, with type'all' | 'smartWalletOnly' | 'eoaOnly' | undefined
.Being able to pass this preference allows app developers better configurability of the Coinbase Wallet popup. Preference based behavior documented here.
Additionally, ahead of Smart Wallet mainnets launch––as of Coinbase Wallet SDK version
4.0.2
(used in Wagmi ^2.9.5)––the Smart Wallet option will no longer show in the Coinbase Wallet popup when the sending app is onlocalhost
. Developers found the host based behavior confusing. Now, if developers using versions^4.0.0
want to see the Smart Wallet in the popup ahead of mainnets launch, they need to passsmartWalletOnly
.