-
-
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
feat: [MC 0.5] - Activity view and Settings on the tab bar #6294
Conversation
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
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.
Code looks good.
when running the app, I had a issue with the spinning circle on activity screen that keeps spinning when you have no tx instead of then displaying "you have no transactions!"
@NicolasMassart Thanks for the review!! Do you remember the network you were in? I think this was already happening, I will take a look |
I was on goerli testnet |
@NicolasMassart I can't reproduce it :/ https://recordit.co/KjHww2kAoa |
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.
Left one comment!
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
… param to false on network selector when navigate to network list on settings
… to be reached for a better ux
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.
Left comments
app/components/Views/Settings/NetworksSettings/NetworkSettings/index.js
Outdated
Show resolved
Hide resolved
…network. Clean code. Solve Bug when exist custom rpc url
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
* Update Test scripts to new Settings tab bar button * Fix return to wallet view screen step * Remove extra steps * Update cucumber tag for android local config files * Update test steps
* Update Test scripts to new Settings tab bar button * Fix return to wallet view screen step * Remove extra steps * Update detox test * Change add custom network test id and update affected files * Fix eslint errors
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
Kudos, SonarCloud Quality Gate passed! |
Description
This PR aims to move the activity view and settings from the drawer view to the bottom tab bar.
Screenshots/Recordings
Scenario: Activity should be a new action that's added to the Tab bar:
Scenario: Settings should be a new action that's added to the Tab bar:
Scenario: Settings screen should show when the Settings tab item is selected: https://recordit.co/esSIk6jVv0
Scenario: Activity screen should show when the Activity tab item is selected: https://recordit.co/RaYkls6qBb
Scenario: Activity and Settings options should be removed from the Drawer
Scenario: Navigate to settings from the tab bar always navigate to Settings view: http://recordit.co/p7ZhSi2bsO
Scenario: Add network, when pressed goes back it always returns to the last screen: https://recordit.co/I1f77adphV
Scenario: Add Custom rpc for ethereum mainnet: https://recordit.co/5DHZTlD7sP
Issue
Progresses #???
Checklist