-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Add Trust Wallet #11566
Add Trust Wallet #11566
Conversation
✅ Deploy Preview for ethereumorg ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@corwintines Are we waiting on anything here? @nhsz Any issues you foresee with this and #12299? |
@corwintines @wackerow thinking again, as we're doing some changes to the wallets metadata in #12299 (eg: some props are being removed), it would be better to wait until that PR is merged, make the necessary changes here and merge it later to avoid conflicts now |
Ok, #12299 has been merged... this should be ready to be updated before merging in |
Description
Related Issue
Fixes #10692