-
-
Notifications
You must be signed in to change notification settings - Fork 280
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
fix(suite-native): unify receive flow navigation for all use cases #16449
Conversation
2ac45c4
to
41bd2ba
Compare
🚀 Expo preview is ready!
|
41bd2ba
to
78facb9
Compare
); | ||
const device = useSelector(selectSelectedDevice); | ||
const hasReceiveButtonRequest = useSelector(hasReceiveAddressButtonRequest); | ||
const ReceiveAccountLoader = () => { |
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.
When can I see this loader?
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.
Not sure, might be dead code. Will discuss with the team and create a follow-up PR if necessary.
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.
Wow, this works really good including Portfolio Tracker and including enabling new coins and adding new accounts in Receive flow. Great job 👏
QA OK Info |
Follow-up for #16312. Once the receive flow is finished (i.e. receive address confirmed), user is returned to the initial screen upon tapping the close button.