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_:sync contact request decision #19938
Conversation
Jenkins BuildsClick to see older builds (12)
|
128a41e
to
430ee6f
Compare
430ee6f
to
803adc3
Compare
Hi @VolodLytvynenko , this PR is ready for manual QA |
77% of end-end tests have passed
Failed tests (10)Click to expandClass TestWalletMultipleDevice:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestDeepLinksOneDevice:
Class TestGroupChatMultipleDeviceMergedNewUI:
Expected to fail tests (2)Click to expandClass TestGroupChatMultipleDeviceMergedNewUI:
Class TestCommunityOneDeviceMerged:
Passed tests (40)Click to expandClass TestCommunityMultipleDeviceMergedTwo:
Class TestActivityMultipleDevicePRTwo:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestCommunityMultipleDeviceMerged:
Class TestCommunityOneDeviceMerged:
Class TestWalletOneDevice:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestActivityMultipleDevicePR:
|
40% of end-end tests have passed
Failed tests (6)Click to expandClass TestDeepLinksOneDevice:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestWalletMultipleDevice:
Passed tests (4)Click to expandClass TestGroupChatMultipleDeviceMergedNewUI:
|
Hi @qfrank thank you for PR. No issues from my side. PR is ready to be merged |
17% of end-end tests have passed
Failed tests (5)Click to expandClass TestWalletMultipleDevice:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestDeepLinksOneDevice:
Passed tests (1)Click to expandClass TestActivityCenterContactRequestMultipleDevicePR:
|
803adc3
to
43d7776
Compare
fixes #19455
after the fix, the result will looks like as screenshot shows
15 Pro is device A and 15 Pro Max is device B in this screenshot.
you can see that there're still differences between A and B after local pair sync.
syncContactRequestForInstallationContact
,syncContactRequestForInstallationContact
will generate AC notification withdefaultContactRequestText
however, as @cammellos said:
as long as the message is the same (i.e accepted on both, declined on both), that's perfectly fine
, I think the result is acceptable now.relate status-go PR
Platforms
status: ready