element-hq / riot-android

A glossy Matrix collaboration client for Android
Apache License 2.0
1.4k stars 394 forks source link

Show contacts from local address book only once (use the android contacts merge) #1346

Open zzottel opened 7 years ago

zzottel commented 7 years ago

Currently, when there are multiple hits at the identity server for one contact, Riot shows multiple lines for this contact. It should be only one line.

This happens if the contact has entered an email address as well as a phone number in Riot, and I have both in my local address book.

It also happens if the same contact has entries in several local address books (contact providers they are called, I think?), e.g. it exists in the Google address book and a CardDAV address book. I have several entries for users where one entry is phone number +49-123-456789 (that's the way I enter the phone numbers myself) and another one is +49123456789. The second entry could possibly come from my WhatsApp account.

ylecollen commented 7 years ago

There is no contact merge i.e each contact field which defines a matrix user id is displayed. It is the expected behaviour.

Some whatApps contacts might be linked to some matrix id so it seems too restricted.

zzottel commented 7 years ago

To clarify: Contacts that only exist in the WhatsApp contact provider should be shown, of course. But (only) in cases where Android auto-merges the contacts from different providers, I'd like to only see one line for the contact.