-
Notifications
You must be signed in to change notification settings - Fork 986
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
[#18964] Support custom message when sending contact request #18998
Conversation
Jenkins BuildsClick to see older builds (23)
|
606e479
to
ee72f73
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
12% of end-end tests have passed
Failed tests (41)Click to expandClass TestGroupChatMultipleDeviceMergedNewUI:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestCommunityOneDeviceMerged:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestActivityMultipleDevicePRTwo:
Class TestActivityMultipleDevicePR:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestCommunityMultipleDeviceMerged:
Expected to fail tests (1)Click to expandClass TestCommunityOneDeviceMerged:
Passed tests (6)Click to expandClass TestDeepLinksOneDevice:
Class TestCommunityOneDeviceMerged:
|
ee72f73
to
a22eac3
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.
🚀
a22eac3
to
b652104
Compare
b652104
to
ebad941
Compare
ebad941
to
68dcb36
Compare
96% of end-end tests have passed
Failed tests (1)Click to expandClass TestCommunityMultipleDeviceMergedTwo:
Expected to fail tests (1)Click to expandClass TestCommunityOneDeviceMerged:
Passed tests (46)Click to expandClass TestGroupChatMultipleDeviceMergedNewUI:
Class TestActivityMultipleDevicePRTwo:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestCommunityOneDeviceMerged:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestDeepLinksOneDevice:
Class TestCommunityMultipleDeviceMerged:
Class TestActivityMultipleDevicePR:
|
fcd3c00
to
6b3b21a
Compare
Hey @status-im/mobile-qa I have a question about this PR. |
Hi @seanstrom, sorry for the late reply! |
Hey @qoqobolo 👋 |
Hi @seanstrom thank you for PR. No issues from my side. PR is ready to be merged |
… sending a contact request
6b3b21a
to
bcfe34e
Compare
fixes #18964
Summary
This PR attempts to resolve the need for a re-frame event handler that can send a contact request with a custom message. This PR builds on the requirements described in #18964 and #18965:
status-im.contexts.chat.contacts.events
Platforms
Areas that maybe impacted
Functional
Steps to test
There are a few steps for setting up the environment, in this case,
Before and after screenshots comparison
Before changes
Send.Contact.Request.Before.mov
After changes
Send.Contact.Request.After.mov
status: ready