-
Notifications
You must be signed in to change notification settings - Fork 985
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
[#21567] - Remove steps to set up profile on new profile creation #21715
Conversation
Jenkins BuildsClick to see older builds (40)
|
1fcd290
to
b473895
Compare
b473895
to
b778e99
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.
LGTM! And thank you for the refactors along the way!
3a9d0f9
to
ed3e170
Compare
0% of end-end tests have passed
Failed tests (8)Click to expandClass TestWalletOneDevice:
Class TestWalletMultipleDevice:
Class TestCommunityOneDeviceMerged:
Class TestCommunityMultipleDeviceMerged:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
|
Hey @ulisesmac ! Thanks for your PR. Found new screen that appeared at Login. ISSUE 1: simplification user creation appeared after sync fallback Precondition: you should have any recovery phrase to use on mobile and to login this user to the desktop as well
Actual result: New simplification user creation appears in this flow. video_2024-12-04_16-53-00.mp4 |
Sorry, I don't get what is wrong, could you please add the "expected result" section. Thank you! Note: I'm not sure if the issue you are pointing out is the current behavior on UpdateI can't replicate the behaviour you showed 🤔 maybe you are performing an additional step? BTW, I can see the onboarding images in your build are the new ones, but the branch in this PR hasn't updated them yet, so maybe the versions tested are different? video: video_2024-12-04_13-48-04.mp4 |
0% of end-end tests have passed
Failed tests (8)Click to expandClass TestCommunityOneDeviceMerged:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestCommunityMultipleDeviceMerged:
Class TestWalletOneDevice:
Class TestWalletMultipleDevice:
|
Hey @ulisesmac ! Thanks for checking. We will fix E2E tests and I let you know. |
62% of end-end tests have passed
Failed tests (3)Click to expandClass TestWalletMultipleDevice:
Class TestCommunityOneDeviceMerged:
Passed tests (5)Click to expandClass TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestCommunityMultipleDeviceMerged:
Class TestWalletOneDevice:
Class TestCommunityOneDeviceMerged:
|
505632a
to
c37bea5
Compare
75% of end-end tests have passed
Failed tests (2)Click to expandClass TestWalletMultipleDevice:
Passed tests (6)Click to expandClass TestWalletOneDevice:
Class TestCommunityOneDeviceMerged:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestCommunityMultipleDeviceMerged:
|
Hi @ulisesmac @mariia-skrypnyk, |
Please, let me know when it is merged and notify in release-coordination channel to cherry-pick it, thanks! |
c37bea5
to
94002c0
Compare
* Remove password disclaimer, update copy and simplify implementation * Remove non-existing wallet event * Skip profile configuration for the "create profile" onboarding flow * Include flow for new profiles when another profile already exists * Add explanation about temp username * Point 20 to a constant * Simplify create profile password screen * Fix password creation screen jump on mount * e2e: updated tests * Completely remove the "create profile" onboarding screen * e2e: updated tests with recovering user --------- Co-authored-by: Yevheniia Berdnyk <ie.berdnyk@gmail.com>
fixes #21567
Summary
Update:
This PR completely removes the "create profile" screen for both "log in" and "create profile" flows.
Testing notes
Please make sure to test all posible flows, such as login, new profile generations when there are other existing profiles, etc.
Platforms
status: ready