Added all platform support for injected Trust Wallet #1357
Added all platform support for injected Trust Wallet #1357chaoticgoodpanda wants to merge 7 commits intothirdweb-dev:v2-web3-onboard-developfrom
Conversation
…at there is a Trust Wallet browser extension.
…at there is a Trust Wallet browser extension.
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
| }) | ||
| }), | ||
| platforms: ['mobile'] | ||
| platforms: ['all'] |
There was a problem hiding this comment.
Is this the only change that occurred with the desktop wallet provider?
There was a problem hiding this comment.
Yes, that's it. Assumption is that's all that's needed to support our new browser extension: https://chrome.google.com/webstore/detail/trust-wallet/egjidjbpglichdcondbcbdnbeeppgdph
There was a problem hiding this comment.
@chaoticgoodpanda great, have you tested with the internal demo project(packages/demo) to verify W3O is working properly with changing of chains, signing, sending transaction, etc?
Running yarn && yarn dev will get it going
Adamj1232
left a comment
Adamj1232
left a comment
There was a problem hiding this comment.
Few changes and a comment for clarification requested
Co-authored-by: Adam Carpenter <adamcarpenter86@gmail.com>
…/web3-onboard into v2-web3-onboard-develop
|
Closing in favor of #1463 |
2 participants
…at there is a Trust Wallet browser extension.
Description
I'm on the Trust team, and we have a Chrome browser extension released: https://chrome.google.com/webstore/detail/trust-wallet/egjidjbpglichdcondbcbdnbeeppgdph. Changing the platform param for Trust's injected wallet in Blocknative to be 'all' instead of 'mobile' so TW extension can be connected via blocknative-powered dApps.
Checklist
package.jsonis incremented following semantic versioningyarn check&yarn buildto confirm there are not any associated errors