-
Notifications
You must be signed in to change notification settings - Fork 78
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(@wallet): multi network history #6271
Conversation
Jenkins BuildsClick to see older builds (13)
|
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.
Good work!
case Constants.networkMainnet: return qsTr("Mainnet"); | ||
case Constants.networkRopsten: return qsTr("Ropsten"); | ||
default: return qsTrId("Unknown") | ||
switch (Web3ProviderStore.chainName) { |
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.
why does this show only these two networks statically? Mainnet and Ropsten? what about the others?
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.
on browser we decided to have only those 2, similar to how it works with ENS/stickers
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 didnt know that !! :D
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.
Awesome work! You made it look so easy!
7523dc5
to
47977ff
Compare
dont merge it yet , lets do after release |
6cee2c1
to
018c036
Compare
018c036
to
c4e3a2d
Compare
Multinetwork:
Remove toggle for multi network
Clean all previous network code
status-go PR: status-im/status-go#2733
fixes #6271
fixes #4749