Skip to content
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

Double Contacts #10

Closed
ghostnegotiator opened this issue Oct 20, 2014 · 9 comments · Fixed by #84
Closed

Double Contacts #10

ghostnegotiator opened this issue Oct 20, 2014 · 9 comments · Fixed by #84
Labels
Milestone

Comments

@ghostnegotiator
Copy link

When adding some one you can get double contacts if he added you back.

img

How to cause it:

What happens: Chaos!

It also affects when you time out from the chat:
img

@ghostnegotiator
Copy link
Author

I noticed, when you "time out" from the client you re sign in it logs you in twice. Which causes this bug...

@zauberstuhl
Copy link

Currently it is not possible to logout with the client. I fixed that with #7 and will be available with the next build. But we should also think about timeout issues like you mentioned.

Thanks 4 reporting

@ghostnegotiator
Copy link
Author

Thanks for notifying, hopefully the build will release soon ;)

@zauberstuhl
Copy link

that should be fixed with #50. please confirm

@Flaburgan
Copy link
Member

It doesn't seem fixed to me on the last develop code.

@zauberstuhl
Copy link

@Flaburgan if you have multiple chat-aspects and one user is in both then you see him twice

@Flaburgan
Copy link
Member

That's my use case. This is still a bug imo.

@zauberstuhl
Copy link

e.g. pidgin does the same thing and it is valid in my opinion. the user is in two categories so he will be displayed in both. we could think about displaying the group name in jsxc. then it should be clear for the user.

@jhass
Copy link
Member

jhass commented Jan 12, 2015

I agree that the current situation is a bug. We either need to display the group name or remove the duplicates. Personally I would tend to do the later.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants