Skip to content

test: member added in channel has display name#8011

Draft
link2xt wants to merge 1 commit intomainfrom
link2xt/kwwmunovwupq
Draft

test: member added in channel has display name#8011
link2xt wants to merge 1 commit intomainfrom
link2xt/kwwmunovwupq

Conversation

@link2xt
Copy link
Collaborator

@link2xt link2xt commented Mar 20, 2026

Test for a problem introduced with #7971
When added contact is synced in channel, it has no display name and as the result second device displays the email address.

@link2xt link2xt force-pushed the link2xt/kwwmunovwupq branch from 5bb21d0 to cd69192 Compare March 20, 2026 07:55
@iequidoo
Copy link
Collaborator

iequidoo commented Mar 20, 2026

Maybe make SetPgpContacts a (Vec<(String, String, String)>) where the last String is contact's authname? And if we already have this contact locally on the 2nd device, update it if the sync message timestamp is bigger

@link2xt
Copy link
Collaborator Author

link2xt commented Mar 20, 2026

Maybe make SetPgpContacts a (Vec<(String, String, String)>) where the last String is contact's authname? And if we already have this contact locally on the 2nd device, update it if the sync message timestamp is bigger

Extending tuple will break compatibility anyway, I wrote more at #7954 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something is not working

2 participants