-
-
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: Send flow UI updates #6802
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. |
629c7b6
to
6fd7a4d
Compare
Rebuilt entirely pipeline with the latest changes: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/c3e17674-c28d-4790-a402-1eadaf3fb1ad |
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 update the send flow UI on this figma file
Also, it was updated the Contacts view on settings was since they shared the same component
This PR also is deleting the recents from the send flow and it's underneath logic
Technical detais
AddressElement, AddressList, and IdentIcon updated to typescript and to functional components.
Deleted recents from the redux state, freeing more memory. Created a migration for the existing users.
Screenshots/Recordings
http://recordit.co/7eR0wBAfbA
E2E build
https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/c3e17674-c28d-4790-a402-1eadaf3fb1ad
Issue
Progresses #???
Checklist