-
Notifications
You must be signed in to change notification settings - Fork 97
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
Replace old wallet with new wallet - Closes #1705 #1736
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.
Thank you, Massao 🥇
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.
🐛 Opening Wallet while non-transfer tx is pending results in error rendering
🐛 2pg reg tx - similar icon as transfer
@slaweet is there a task to restyle latest activity widget?
@Efefefef "Latest activity" widget on Dashboard will be updated in a few sprints when we do the new UX of Dashboard. |
What issue have I solved?
#1705
How have I implemented/fixed it?
Deleted
walletTransactions
folder, which had the previous wallet page, didn't delete the transactions file, since the old ones are still being used on some parts of the application.Updated initial transactions load to load 30 transactions, also show more loads 30 more each time.
Updated
/wallet
to point to the new wallet, with new header and transaction table.Adjusted e2e tests where needed, due to the changes on behavior.
How has this been tested?
On
/wallet
it should have the new header and transactions table with load more button, clicking on a transaction should take the user to/explorer/transactions
instead of how it was before.On
/dashboard
clicking on a transaction should also take the user to the/explorer/transactions
.The
/explorer/account
, should keep the old table and the infinite scrolling, to load more.Review checklist