-
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
Implement Contact Request flow in Contact Profile #19039
Conversation
Jenkins BuildsClick to see older builds (52)
|
50158d0
to
913963f
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.
Self Review / Comments
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, LGTM.
802d9c0
to
c043244
Compare
23f9b9c
to
fc17a3d
Compare
17% of end-end tests have passed
Failed tests (39)Click to expandClass TestDeepLinksOneDevice:
Class TestCommunityMultipleDeviceMerged:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestActivityMultipleDevicePR:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestActivityMultipleDevicePRTwo:
Expected to fail tests (1)Click to expandClass TestCommunityOneDeviceMerged:
Passed tests (8)Click to expandClass TestCommunityOneDeviceMerged:
Class TestActivityCenterContactRequestMultipleDevicePR:
|
fc17a3d
to
be23df4
Compare
be23df4
to
d386e78
Compare
88% of end-end tests have passed
Failed tests (5)Click to expandClass TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestCommunityMultipleDeviceMerged:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestGroupChatMultipleDeviceMergedNewUI:
Expected to fail tests (1)Click to expandClass TestCommunityOneDeviceMerged:
Passed tests (42)Click to expandClass TestCommunityOneDeviceMerged:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestCommunityMultipleDeviceMerged:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestDeepLinksOneDevice:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestActivityMultipleDevicePRTwo:
Class TestActivityMultipleDevicePR:
|
db4dca0
to
ef88c00
Compare
ef88c00
to
46edb23
Compare
46edb23
to
0a24f3d
Compare
@seanstrom thanx for the PR! Does this PR require manual testing? Is anything could be affected in app if feature flag is disabled? Are |
Hey @pavloburykh 👋 The changes to the top-drawer component will be active without the feature flag, and I think we can find examples of it being used by looking for features that use the drawer / bottom-sheet UI. I looked through the code and it seems there's some usages that can be found while requesting to join a community. Screen.Recording.2024-03-05.at.16.59.43.movThere could be some more examples that are worth double-checking and I'm not sure where all of them are located, but I can help find some other places by looking through the code some more. Lmk if we need some more examples 🙏 |
@seanstrom thank you! PR is ready for merge. |
0a24f3d
to
56851b8
Compare
…nside drawer-top component
56851b8
to
75dce68
Compare
fixes #18965
aligns with #18998
Summary
This PR attempts to implement the new designs for the Contact Request in Contact Profiles. This PR is dependent on the changes inside PR #18998, which means that contact requests will not support personalised messages until those changes are merged. This PR aligns with the changes inside #18998, but does directly rely on those commits in this branch.
Testing notes
Note that while testing this PR the personalised messages will not be functioning properly until [#18964] Support custom message when sending contact request #18998 is merged.All other functionality related to Feature: New contact request user flow #18965 is intended to be functioning correctly.top-drawer
component, which may affect other UI elements like bottom sheets or drawers.Platforms
Areas that maybe impacted
Functional
Steps to test
new-contact-ui
feature flag in the mobile appBefore and after screenshots comparison
Before
After
Contact.Request.After.mov
status: ready