zulip-terminal icon indicating copy to clipboard operation
zulip-terminal copied to clipboard

Two users with the the same name appear as the same person

Open lzmartinico opened this issue 3 years ago • 24 comments

On zulip-term version 0.6.0, if two users have the same name and send two messages in sequence within the same topic, the messages will be displayed as if coming from the same user

lzmartinico avatar Feb 11 '22 19:02 lzmartinico

@lzmartinico Thanks for reporting this! I suspect this is present in main also, though have not confirmed.

This should be fairly simple to resolve via comparing user ids rather than names.

neiljp avatar Feb 14 '22 06:02 neiljp

Glad to hear there is an easy fix! Maybe there should be another ticket (or meta ticket), but support for users with the same name is somewhat lacking overall (e.g. when tagging a user through autocompletion, it's impossible to know which one is selected)

lzmartinico avatar Feb 14 '22 15:02 lzmartinico

@lzmartinico Autocomplete is improved further in the current development version after GSoC, and may well resolve the issue you're describing.

Generally if you have other separate then other issues (or a general issue we can split out) are a good idea - or if you wish to discuss a potential range of issues then it's likely better to raise this in a topic in the #zulip-terminal stream on chat.zulip.org :)

neiljp avatar Feb 14 '22 18:02 neiljp

@lzmartinico thank you for the feedback!

We have additional autocomplete improvements for tagging users (disambiguation using an appended user ID, #928 for reference), as well as using users' names as private message recipients, currently on the main branch. It will be a part of the next release, but you can also pull that from main as the latest installation as mentioned on our README.

Any additional feedback is highly appreciated!

prah23 avatar Feb 14 '22 18:02 prah23

Hi all, I would like to take up this issue. @zulipbot claim

Dishti-Oberai avatar Apr 28 '22 06:04 Dishti-Oberai

Hello @Dishti-Oberai!

Thanks for your interest in Zulip! You have attempted to claim an issue without the label "help wanted". You can only claim and submit pull requests for issues with the help wanted label.

If this is your first time here, we recommend reading our guide for new contributors before getting started.

zulipbot avatar Apr 28 '22 06:04 zulipbot

@Dishti-Oberai You have been unassigned from this issue because you have not made any updates for over 14 days. Please feel free to reclaim the issue if you decide to pick up again. Thanks!

zulipbot avatar May 09 '22 07:05 zulipbot

I am still working on it

Dishti-Oberai avatar May 09 '22 12:05 Dishti-Oberai

Hi is this still an issue? How would you solve the problem of the names being same? Displaying the ids with the names in the sender name instead of just the name? Screenshot 2022-08-28 at 6 33 48 PM

Sushmey avatar Aug 28 '22 13:08 Sushmey

@Sushmey Thanks for your interest in the project! To clarify based on your doing a capture of the zulip web app, this issue tracker is for zulip-terminal, not the zulip web app. If there is an issue there, then it needs resolving separately, though from the screenshot it seems that if the user profile picture is different then that is one way to show it (or cause the problem). Manually one can also look at the senders, but that's not preferred I expect.

For zulip-terminal, perhaps obviously, we don't have user profiles to distinguish, which I expect is why you suggested the ids?

The other current issue here is that the same 'A0' header (in your case) will be used in ZT for both messages, so it really looks like the messages are from the same user - the sender header is not even repeated. That was the change I was planning to address first. Once we have that, then we'll also be able to distinguish adjacent messages, so could consider adjusting the sender somehow. That'd need to be carefully done, to avoid indicating that the sender name was actually different.

neiljp avatar Aug 28 '22 15:08 neiljp

Oh thank you for pointing that out! I was under the impression that the web app was the one where the problem could be recreated but it's only where you discuss the problems.

I see that someone in that someone has already solved it? Please let me know so I can try my hand at it https://chat.zulip.org/#narrow/stream/206-zulip-terminal/topic/.23T1151.20Users.20with.20same.20name.20appear.20as.20same.20person

Sushmey avatar Aug 28 '22 15:08 Sushmey

@zulipbot claim

Sushmey avatar Sep 13 '22 14:09 Sushmey

Welcome to Zulip, @Sushmey! We just sent you an invite to collaborate on this repository at https://github.com/zulip/zulip-terminal/invitations. Please accept this invite in order to claim this issue and begin a fun, rewarding experience contributing to Zulip!

Here's some tips to get you off to a good start:

As you work on this issue, you'll also want to refer to the Zulip code contribution guide, as well as the rest of the developer documentation on that site.

See you on the other side (that is, the pull request side)!

zulipbot avatar Sep 13 '22 14:09 zulipbot

@Sushmey You have been unassigned from this issue because you have not made any updates for over 14 days. Please feel free to reclaim the issue if you decide to pick up again. Thanks!

zulipbot avatar Nov 21 '22 06:11 zulipbot

@zulipbot still working on it...

Sushmey avatar Nov 21 '22 16:11 Sushmey

Hi @neiljp I want to work on this issue, can you please assign it to me? Basically i have to make sure that users having same name when message in sequence, their messages do no come under same topic. Right? Please give me more info if I lack somewhere...

SRV332003 avatar Dec 13 '22 10:12 SRV332003

Hi @SRV332003 I am currently working on this issue with an active PR. If you wish to contribute to the repository, you may work on some other issue.

Sushmey avatar Dec 13 '22 11:12 Sushmey

@Sushmey You have been unassigned from this issue because you have not made any updates for over 14 days. Please feel free to reclaim the issue if you decide to pick up again. Thanks!

zulipbot avatar Apr 03 '23 08:04 zulipbot

@Sushmey You have been unassigned from this issue because you have not made any updates for over 14 days. Please feel free to reclaim the issue if you decide to pick up again. Thanks!

zulipbot avatar May 05 '23 10:05 zulipbot

Welcome to Zulip, @Chan-Dong-Jun! We just sent you an invite to collaborate on this repository at https://github.com/zulip/zulip-terminal/invitations. Please accept this invite in order to claim this issue and begin a fun, rewarding experience contributing to Zulip!

Here's some tips to get you off to a good start:

As you work on this issue, you'll also want to refer to the Zulip code contribution guide, as well as the rest of the developer documentation on that site.

See you on the other side (that is, the pull request side)!

zulipbot avatar Jan 30 '24 22:01 zulipbot

@Sushmey You have been unassigned from this issue because you have not made any updates for over 14 days. Please feel free to reclaim the issue if you decide to pick up again. Thanks!

zulipbot avatar Apr 11 '24 02:04 zulipbot