-
Notifications
You must be signed in to change notification settings - Fork 987
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: remove contact from mutual contact list after contact is blocked #15579
Conversation
Jenkins BuildsClick to see older builds (43)
|
f3d7670
to
10b29bc
Compare
(assoc :blocked? true | ||
:added? false)) | ||
:added? false | ||
:active? false)) |
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.
Should we set the mutual?
attribute to false
too ?
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.
@ibrkhalil does not seem to have any effect but it can be set to false since the two contacts in this context are no longer mutual the moment one becomes blocked
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.
yeah you are right
10b29bc
to
afc10bd
Compare
Hi @jo-mut let me know please if this PR is ready for tests |
afc10bd
to
bdef700
Compare
@VladimrLitvinenko this is ready you can proceed with the testing. Thanks |
86% of end-end tests have passed
Failed tests (4)Click to expandClass TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestCommunityMultipleDeviceMerged:
Passed tests (25)Click to expandClass TestActivityMultipleDevicePR:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestCommunityOneDeviceMerged:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestCommunityMultipleDeviceMerged:
|
bdef700
to
62ad5a5
Compare
@jo-mut thanx for the PR. Initial issue misses the case with contact removal after which contact list is not being updated too. Is it possible to cover contact removal case by this PR? ISSUE 1 Removed contact is shown in contact list until app reopening or opening removed user profile screenPreconditions: User A and User B are contacts Steps:
Actual result: User B remains in User's A contact list until app reopening or opening removed user profile screen telegram-cloud-document-2-5307848119155697470.mp4 |
90% of end-end tests have passed
Failed tests (3)Click to expandClass TestCommunityMultipleDeviceMerged:
Class TestActivityCenterContactRequestMultipleDevicePR:
Passed tests (26)Click to expandClass TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestCommunityMultipleDeviceMerged:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestCommunityOneDeviceMerged:
Class TestActivityMultipleDevicePR:
Class TestActivityCenterContactRequestMultipleDevicePR:
|
@pavloburykh currently this pr as is removes User B from User's A list of contacts. I will update the pr so that user B is also removed from user A list contact. Screen_Recording_20230411_154758_Status.Debug.mov |
4c84372
to
96e9ab5
Compare
@pavloburykh I have updated the pr so that both contact are removed from each others list but just before you run the tests I would like to take another look at status-go please hold on running tests on this for a while |
Also please note this behavior now: currently after blocking/removing a user, it disappears from the contact list only after reopening this list. Thank you! |
701a53c
to
00a3aee
Compare
@pavloburykh this is ready. Thanks |
hey @jo-mut could you please rebase and more to e2e |
00a3aee
to
2a3f9ef
Compare
6797684
to
abf1cf0
Compare
93% of end-end tests have passed
Failed tests (2)Click to expandClass TestActivityCenterContactRequestMultipleDevicePR:
Class TestCommunityMultipleDeviceMerged:
Passed tests (27)Click to expandClass TestCommunityOneDeviceMerged:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestActivityMultipleDevicePR:
Class TestCommunityMultipleDeviceMerged:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestActivityCenterContactRequestMultipleDevicePR:
|
abf1cf0
to
9eef676
Compare
hi @jo-mut, thank you for PR. no issues from my side. PR is ready to be merged |
9eef676
to
340ce04
Compare
fixes #15279
Summary
This pr removes user A (the blocked contact) from the contact list of user B once blocked and removes user B from user A list of contacts