-
Notifications
You must be signed in to change notification settings - Fork 984
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 account edit confirmation button #18930
Conversation
Jenkins BuildsClick to see older builds (34)
|
any chance of a photo/video to see this in action? |
Added to the description |
src/status_im/contexts/wallet/common/screen_base/create_or_edit_account/view.cljs
Outdated
Show resolved
Hide resolved
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 👍
src/status_im/contexts/wallet/common/screen_base/create_or_edit_account/view.cljs
Outdated
Show resolved
Hide resolved
src/status_im/contexts/wallet/common/screen_base/create_or_edit_account/view.cljs
Outdated
Show resolved
Hide resolved
5aff7e7
to
4fb7218
Compare
12% of end-end tests have passed
Failed tests (41)Click to expandClass TestCommunityOneDeviceMerged:
Class TestCommunityMultipleDeviceMerged:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestActivityMultipleDevicePR:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestActivityMultipleDevicePRTwo:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestCommunityMultipleDeviceMergedTwo:
Expected to fail tests (1)Click to expandClass TestCommunityOneDeviceMerged:
Passed tests (6)Click to expandClass TestCommunityOneDeviceMerged:
Class TestDeepLinksOneDevice:
|
(rn/dismiss-keyboard!) | ||
(save-account {:account account | ||
:updated-key :name | ||
:new-value @edited-account-name}))] | ||
(fn [] | ||
(let [{:keys [name emoji address color watch-only? default-account?] |
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.
nit: we can destructure name as wallet-name to avoid shadowing clojure.core/name
4fb7218
to
f1f286f
Compare
Thanks for the fixes @clauxx! |
f1f286f
to
6c5706b
Compare
Signed-off-by: Cristian Lungu <lungucristian95@gmail.com>
Signed-off-by: Cristian Lungu <lungucristian95@gmail.com>
Signed-off-by: Cristian Lungu <lungucristian95@gmail.com>
Signed-off-by: Cristian Lungu <lungucristian95@gmail.com>
Signed-off-by: Cristian Lungu <lungucristian95@gmail.com>
Signed-off-by: Cristian Lungu <lungucristian95@gmail.com>
Signed-off-by: Cristian Lungu <lungucristian95@gmail.com>
Signed-off-by: Cristian Lungu <lungucristian95@gmail.com>
Signed-off-by: Cristian Lungu <lungucristian95@gmail.com>
Signed-off-by: Cristian Lungu <lungucristian95@gmail.com>
Signed-off-by: Cristian Lungu <lungucristian95@gmail.com>
43391cc
to
7da7be2
Compare
fixes #18834
fixes #18791
Summary
On the account edit screen, always show the confirmation button, even if the input is focused. Also, fixed the issue with the text in the button.
Platforms
Functional
Steps to test
Demo
RPReplay_Final1708522337.2.MP4
status: ready