-
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: slide button #18922
Fix: slide button #18922
Conversation
Jenkins BuildsClick to see older builds (31)
|
8c10e64
to
7ba9700
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 one @OmarBasem.
Minor detail, as you have this shown in the video attached but the description should mention that this affects the create account page and should be tested there 👍
90% of end-end tests have passed
Failed tests (4)Click to expandClass TestActivityMultipleDevicePRTwo:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestGroupChatMultipleDeviceMergedNewUI:
Expected to fail tests (1)Click to expandClass TestCommunityOneDeviceMerged:
Passed tests (43)Click to expandClass TestCommunityMultipleDeviceMerged:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestDeepLinksOneDevice:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestCommunityOneDeviceMerged:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestActivityMultipleDevicePR:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestCommunityMultipleDeviceMergedTwo:
|
b960b49
to
4746535
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 @OmarBasem !
Just one question, in the video, at the end, some schema errors are shown, are they related to this PR?
Nope not related. I will try to see where they come from and open an issue |
4746535
to
869aae2
Compare
Hi @OmarBasem ! Thanks for your PR!
I have an issue: ISSUE 1: Slide button on Create Account is enabled if user didn't enter account name Steps:
Actual result: slide button is active video_2024-02-29_13-29-56.mp4Expected result: slide button should be disabled |
869aae2
to
dde815d
Compare
15a9f90
to
4d3a303
Compare
@mariia-skrypnyk thanks for your testing! It should be working as expected now |
e2c9b70
to
a36ea56
Compare
✔️ status-mobile/prs/android-e2e/PR-18922#11 🔹 ~8 min 3 sec 🔹 a36ea56 🔹 📦 android-e2e package |
Thank you @mariia-skrypnyk for your testing and sorry for not noticing this. It is fixed now! |
85% of end-end tests have passed
Failed tests (6)Click to expandClass TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestCommunityMultipleDeviceMerged:
Expected to fail tests (1)Click to expandClass TestCommunityOneDeviceMerged:
Passed tests (41)Click to expandClass TestGroupChatMultipleDeviceMergedNewUI:
Class TestActivityMultipleDevicePRTwo:
Class TestCommunityMultipleDeviceMerged:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestCommunityOneDeviceMerged:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestDeepLinksOneDevice:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestActivityMultipleDevicePR:
|
c97be55
to
62eafef
Compare
Hi @OmarBasem ! |
fixes: #18894
This PR fixes the
disabled?
prop for the slide button where the slide button wouldn't slide after changingdisabled?
from true to falseParts affected:
Screen_Recording_20240221_114329_Status.mp4