-
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
Add support for discovering local pairing peers #16803
Conversation
Jenkins BuildsClick to see older builds (8)
|
3f78f2f
to
d3e3f51
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! These logs will help us a lot. 🚀
d3e3f51
to
26876b3
Compare
88% of end-end tests have passed
Failed tests (5)Click to expandClass TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestCommunityOneDeviceMerged:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestActivityCenterContactRequestMultipleDevicePR:
Passed tests (35)Click to expandClass TestActivityMultipleDevicePR:
Class TestCommunityOneDeviceMerged:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestCommunityMultipleDeviceMerged:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
|
@status-im/mobile-qa : This PR does not require testing, I ran E2E, is it okay to merge now? |
sure, thank you for asking! |
This commit initialises the status-go method `startSearchForLocalPairingPeers` which in turn will produce logs that are important for peer discovery while local pairing and will produce logs important to detect local pairing crashes.
26876b3
to
0cc40fe
Compare
Summary
This PR initialises the status-go method
startSearchForLocalPairingPeers
which in turn will produce logs that are important for peer discovery while local pairing and will produce logs important to detect local pairing crashes.Platforms
Functional
Steps to test
This PR does not require any testing.
status: ready