-
Notifications
You must be signed in to change notification settings - Fork 992
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
Add status tag for pending contact request to contact profile #19075
Add status tag for pending contact request to contact profile #19075
Conversation
Jenkins BuildsClick to see older builds (24)
|
2b70a02
to
2851a83
Compare
2851a83
to
7873d33
Compare
40% of end-end tests have passed
Failed tests (28)Click to expandClass TestCommunityMultipleDeviceMergedTwo:
Class TestCommunityMultipleDeviceMerged:
Class TestActivityMultipleDevicePR:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Expected to fail tests (1)Click to expandClass TestCommunityOneDeviceMerged:
Passed tests (19)Click to expandClass TestCommunityMultipleDeviceMergedTwo:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestCommunityOneDeviceMerged:
Class TestDeepLinksOneDevice:
Class TestActivityMultipleDevicePRTwo:
|
7873d33
to
338457f
Compare
94% of end-end tests have passed
Failed tests (2)Click to expandClass TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestGroupChatMultipleDeviceMergedNewUI:
Expected to fail tests (1)Click to expandClass TestCommunityOneDeviceMerged:
Passed tests (45)Click to expandClass TestDeepLinksOneDevice:
Class TestCommunityMultipleDeviceMerged:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestActivityMultipleDevicePRTwo:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestCommunityOneDeviceMerged:
Class TestActivityMultipleDevicePR:
|
9f7aca1
to
cc5d226
Compare
@status-im/mobile-qa Can you please review the E2E results 🙏 |
@seanstrom thanx for the PR. e2e failures are not PR related. Ready for merge. |
cc5d226
to
fa0f025
Compare
fa0f025
to
954b1d6
Compare
fixes #19035
Summary
This PR attempts to add a status-tag for representing a pending contact request on the contact profile. Note, that this PR does not hide the "Send contact request" button. This separation of functionality is to avoid some conflicts with this PR #19039, which will be responsible for hiding the "Send contact request" button after sending a contact request.
Platforms
Areas that maybe impacted
Functional
Steps to test
new-contact-ui
feature flag is disabled on mobilenew-contact-ui
feature flag on mobileBefore and after screenshots comparison
Before
Screen.Recording.2024-03-01.at.14.32.32.mov
After
Screen.Recording.2024-03-01.at.14.37.17.mov
status: ready