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
[DDW-694] Implement the Tokens screen #2671
Conversation
Thanks, @gabriela-ponce. They are all done, except for the last one:
✅ ✅ ✅
✅
✅ Hiding a tab is not good, as it would jump from one wallet to the other, but I have added the same "no tokens" screen as the Wallet Summary
We are not implementing the "recently used tokens" list, for now, so in this case you mentioned, the received token is in the complete list. However, I have found a strange behavior: when sending a token, the receiver receives all tokens the sender has, with zero amount. I'm not sure if this is expected, but it looks strange: The tx lists only the token that was actually sent: |
@daniloprates The fixes look good. Will continue tomorrow with a few more tests in different platforms before approving 👍 |
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.
Loogs good)
@alexander-rukin nice catch! Fixed it. |
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.
All fine, just check my comments
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.
Great job Danilo!
This PR implements the wallet Tokens dedicated screen.
Todos
Screenshots
Testing Checklist
Test Scenarios
Scenario 1 - Validate native token list on the summary screen
Scenario 2 - Validate summary screen for a wallet without Native Tokens transactions
Scenario 3 - Access "Tokens" tab from "Summary"
Scenario 4 - Validate native token tab when wallet is synced
Scenario 5 - Select a token as favorite on "Tokens"
Scenario 6 - Select a token as favorite on "Summary"
Scenario 7 - Remove a token from favorites on "Tokens"
Scenario 8 - Remove a token from favorites on "Summary"
Scenario 10 - Validate token view for a syncing wallet
Review Checklist
Basics
feature
/bug
/chore
,release-x.x.x
)yarn test
)yarn dev
)yarn package
/ CI builds)yarn flow:test
)yarn lint
)yarn prettier:check
)yarn manage:translations
produces no changes)yarn storybook
)yarn.lock
file is updatedCode Quality
Testing
After Review
done
column on the YouTrack board