-
Notifications
You must be signed in to change notification settings - Fork 992
🚀 contacts performance 🚀 #12643
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
🚀 contacts performance 🚀 #12643
Conversation
Jenkins BuildsClick to see older builds (11)
|
90% of end-end tests have passed
Failed tests (7)Click to expand
Passed tests (61)Click to expand |
@flexsurfer |
1d98a05
to
2d7a4b1
Compare
@churik done |
91% of end-end tests have passed
Failed tests (6)Click to expand
Passed tests (61)Click to expand |
@flexsurfer thank you for PR! ISSUE 1: unblocking and removing the user from contacts is not synced between paired devicesSteps:
Expected result: Actual result: OS: IOS, Android ISSUE 2: Users, who are not added to contacts, don't appear in mention suggestion list |
unfortunately its a price for now, but anyway we want to remove public chats soon, for communities and group chats there should be another solution implemented separately |
@cammellos @hesterbruikman |
My 2p is that it would be frustrating for user if they ever have to mention someone, so I'd be inclined in fixing it, communities I'd be ok as still in alpha, but public/group chats he would be noticeable |
i'll fix that then, thanks |
2d7a4b1
to
47de8f2
Compare
thanks @churik all fixed |
Awesome work @flexsurfer! |
47de8f2
to
e58191f
Compare
96% of end-end tests have passed
Failed tests (3)Click to expand
Passed tests (64)Click to expand
|
33% of end-end tests have passed
Failed tests (2)Click to expand
Passed tests (1)Click to expand
|
19c455a
to
dcc62ec
Compare
97% of end-end tests have passed
Failed tests (2)Click to expand
Passed tests (65)Click to expand
|
dcc62ec
to
1bbebc9
Compare
🚀 contacts performance 🚀
replaced systemTags on added and blocked fields, all other changes in status-go
QA: pls ping me when testing