-
Notifications
You must be signed in to change notification settings - Fork 979
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
[#19946] feat: add key pair QR code view #20073
Conversation
Jenkins BuildsClick to see older builds (17)
|
4b900ec
to
c60af0d
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.
Awesome work! 🙌
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/encrypted_qr/view.cljs
Outdated
Show resolved
Hide resolved
b25216b
to
2c6899d
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.
Nice work! 🙌
79% of end-end tests have passed
Failed tests (9)Click to expandClass TestWalletOneDevice:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestWalletMultipleDevice:
Class TestCommunityMultipleDeviceMerged:
Expected to fail tests (2)Click to expandClass TestCommunityOneDeviceMerged:
Class TestGroupChatMultipleDeviceMergedNewUI:
Passed tests (41)Click to expandClass TestCommunityOneDeviceMerged:
Class TestWalletOneDevice:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestDeepLinksOneDevice:
Class TestActivityMultipleDevicePR:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestActivityMultipleDevicePRTwo:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestCommunityMultipleDeviceMerged:
|
Hey @status-im/mobile-qa could you please check test results? |
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.
Well done! ✅
hi @mohsen-ghafouri, thanks for your PR, e2e failures are known and not related. But quick manual testing would be nice to have here |
Hi @yevh-berdnyk this screen is still behind the feature flag, I think we agreed to have manual QA when we want to remove feature flag. |
@mohsen-ghafouri thanks for clarification. In this case we can skip manual QA. PR is ready for merge. |
2c6899d
to
d8efb87
Compare
d8efb87
to
c59bebd
Compare
fixes #19946
Summary
Implement the action for displaying the QR code for a key-pair inside wallet settings.
Areas that maybe impacted
Steps to test
Result
Simulator.Screen.Recording.-.iPhone.15.Pro.Max.-.2024-05-17.at.17.01.55.mp4
status: ready