-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Improvement/assets by chainid #2441
Conversation
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.
Looks like the migration didn't get picked up
For example, OMI should not be here under BNB as it belongs to GoChain network
Steps:
- build v2.0.1 on both OS’s
- yarn clean && yarn watch:clean
- yarn start:ios and yarn start:android
- import your wallet, add networks + tokens
- stay on either network (before putting app in background stay on BNB or GoChain)
- put app in background
- switch to improvement/assets-by-chainid
- yarn clean && yarn watch:clean
- yarn start:ios and yarn start:android
- login, notice how it’s still the same
This does work when you create a fresh wallet and add the networks and custom tokens seen here = http://recordit.co/f7zWGgcDIh |
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.
QA Passed 👍🏽
…etamask-mobile into improvement/assets-by-chainid
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!
* migrations * add custom rpc assets * addpackage * bumpcontroller
Description
Deprecation of
networkType
in favor ofchainId
to store assets.A migration was added to migrate existing assets
Checklist
Issue
Resolves #2391