-
Notifications
You must be signed in to change notification settings - Fork 987
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
[#20165] feat: import a missing key pair by scanning QR code #20280
Conversation
Jenkins BuildsClick to see older builds (12)
|
a14dc95
to
92ba1ee
Compare
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.
Looks good ✅
Left one tiny comment about an optional memo
src/status_im/contexts/settings/wallet/keypairs_and_accounts/actions/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.
🚀
src/status_im/contexts/settings/wallet/keypairs_and_accounts/actions/view.cljs
Outdated
Show resolved
Hide resolved
src/status_im/contexts/settings/wallet/keypairs_and_accounts/actions/view.cljs
Outdated
Show resolved
Hide resolved
45% of end-end tests have passed
Not executed tests (1)Failed tests (25)Click to expandClass TestActivityMultipleDevicePR:
Class TestCommunityOneDeviceMerged:
Class TestWalletMultipleDevice:
Class TestActivityMultipleDevicePRTwo:
Class TestCommunityMultipleDeviceMerged:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestDeepLinksOneDevice:
Expected to fail tests (3)Click to expandClass TestGroupChatMultipleDeviceMergedNewUI:
Class TestWalletOneDevice:
Class TestCommunityOneDeviceMerged:
Passed tests (23)Click to expandClass TestCommunityOneDeviceMerged:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestWalletOneDevice:
|
ef4b0f8
to
187000a
Compare
92% of end-end tests have passed
Not executed tests (1)Failed tests (2)Click to expandClass TestWalletMultipleDevice:
Expected to fail tests (2)Click to expandClass TestCommunityOneDeviceMerged:
Class TestWalletOneDevice:
Passed tests (47)Click to expandClass TestActivityMultipleDevicePR:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestWalletOneDevice:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestCommunityOneDeviceMerged:
Class TestDeepLinksOneDevice:
Class TestActivityMultipleDevicePRTwo:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestCommunityMultipleDeviceMerged:
|
@status-im/mobile-qa would you mind check test result for this PR? |
Hi @mohsen-ghafouri ! |
187000a
to
66b14d3
Compare
fixes #20165
Summary
Implement the UI and UX flow inside the wallet settings for scanning a missing key pair's QR code.
Areas that maybe impacted
Test note
To have a missing key pair on Device B, after you pair devices you can add new keypair on Device A and the on device B logout/login again. you will see missing keypair on device B
Steps to test
Result
2024-05-31.11.31.38.mp4
status: ready