-
Notifications
You must be signed in to change notification settings - Fork 8
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
Update Network Management #215
Conversation
@h0ngcha0 Could you add screenshots to this so I could review it without running the code? |
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.
LTGM 👍
@h0ngcha0 It seems the reset icon is not aligned with the text next to it |
Thanks, fixed now |
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 👍
LGTM. Let's release |
Mainnet
(already possible toTestnet
andDevnet
)devnet
,testnet
ormainnet
network. b) tokens in token list belong to one oftestnet
ormainnet
. Due to these, custom network feature is probably not used that much.nodeAPIKey
000301
NFT since explorer backend2.2.1
is now used.There is already a button to reset to default network if user messes up the network settings.