-
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: add spacing between wrong-password error and forgot-password link #21594
base: develop
Are you sure you want to change the base?
Conversation
Jenkins BuildsClick to see older builds (23)
|
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.
Thanks for fixing it, it annoys me a lot!
6996e36
to
16e4093
Compare
88% of end-end tests have passed
Expected to fail tests (1)Click to expandClass TestCommunityMultipleDeviceMerged:
Passed tests (7)Click to expandClass TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestWalletOneDevice:
Class TestWalletMultipleDevice:
Class TestCommunityOneDeviceMerged:
|
16c6c29
to
551c8c4
Compare
Hey @seanstrom ! Do you need any help from QA side? |
551c8c4
to
a1425c9
Compare
Hey @mariia-skrypnyk 👋 I think I just need a quick check on e2e tests, I think my changes should be unrelated to any failures. |
Hi @seanstrom, e2e failure is not PR related, thanks! |
a1425c9
to
167575f
Compare
@Francesca-G I think this PR is ready for a design-review 🙏 |
fixes #21207
Summary
Platforms
Areas that maybe impacted
Functional
Steps to test
Before and after screenshots comparison
After changes
status: ready