-
Notifications
You must be signed in to change notification settings - Fork 985
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
Features/keep contacts in sync #6697
Conversation
Pull Request Checklist
|
af917c5
to
f25e41f
Compare
75% of end-end tests have passed
Failed tests (15)Click to expand
Passed tests (45)Click to expand |
f25e41f
to
16b0fc9
Compare
@churik latest build does not show your own messages as new (the same in public chat), it was a regression introduced by a refactoring PR. |
@cammellos i have noticed sync is not being done in the following case:
Actual result: contact added on step 8 is not synced on desktop and not present there in contact list. Could you please take a look? cc @churik |
@asemiankevich sorry just to make sure, sync it's a one directional thing. I am assuming all devices are paired & enabled here. So when you click on Here it seems If you click on Is that the case or I misunderstood? |
yes, that is correct @cammellos and i tapped also 1 more question - i need to sync devices manually all the time i add new contacts/ chats across devices? |
@asemiankevich no, once paired, they are kept in sync, but it works in the same way as messages, so if for example one device is offline for too long and the message is not on the mailserver, that contact won't be synced etc. |
@cammellos i have created group chat from mobile device with you. Added you as a contact on ios. Android shows me you in contacts too (good). Desktop does not show either group chat or you as a contact. |
@asemiankevich ping me on status or here when you have 10 mins, I would like to try out something, thanks! |
@asemiankevich I followed your steps and between 14 and 15 should be one step more: tap on "Sync all" from IOS or Android devices, because new device (desktop) was added. Tried to reproduce from scratch 2 times, and I always have new contact on desktop after I tap on IOS or Android "Sync all" (after 14 step). But if you say that you tapped on IOS (or Android) "Sync all" after 14 step and nothing appears on desktop - it is device specific or contact-specific and only @cammellos can help |
@churik i spoke with @cammellos and seems we can merge with this , it is very hard to repro now and definitely not crucial. We can report it later once i can repro it 100% |
Automated tests failures are unrelated to PR.
Aren't synced:
Known behavior:
|
And we can close #6658 I think |
397e17e
to
b5fd67e
Compare
Everytime a contact request is sent/confirmed a sync message is also sent to other devices so the contact is kept in sync. Signed-off-by: Andrea Maria Piana <andrea.maria.piana@gmail.com>
b5fd67e
to
1104bec
Compare
Summary:
Once two devices are paired, contacts are kept in sync by sending an updated sync message every time we send/accept a contact request.
Areas that maybe impacted (optional)
Functional
Only the last commit needs reviewing
status: ready