-
Notifications
You must be signed in to change notification settings - Fork 19
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
added change network method #177
added change network method #177
Conversation
chescalante
commented
Sep 30, 2021
275fe17
to
0deb443
Compare
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.
I would expand a little bit further on this feature:
- Metamask -> allows to switch via RPC
- Nifty/Liquiality -> displays message "Please change the network in your wallet"
- Wallet Connect -> depends on the device. Displays message "Please configure the network in your device"
- Torus and hardware wallets -> "Do you want to disconnect your device and connect it to another network?" and button does disconnect and go to step choose network
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.
This looks good. I agree with the comments from Ilan and would add that "RSK Testnet" should not have a gray color, it looks disabled.
Let's create a new issue for these and perhaps bounties?
* added modal to show wallet information * mock TextEncoder * review changes * fix tests * fix cypress tests * added REACME docs about showModalInfo * changed showModalInfo to showWalletInfo and refactor readme doc * added change network method * added showChangeNetwork doc * minor change Co-authored-by: Christian Escalante <chescalante@gmail.com> Co-authored-by: Ilan <36084092+ilanolkies@users.noreply.github.com>