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

Deactivated Accounts still show up in Contacts for Chat and Videocalls #5608

Closed
RuudschMaHinda opened this issue Jul 4, 2017 · 5 comments · Fixed by #10543
Closed

Deactivated Accounts still show up in Contacts for Chat and Videocalls #5608

RuudschMaHinda opened this issue Jul 4, 2017 · 5 comments · Fixed by #10543

Comments

@RuudschMaHinda
Copy link
Member

Opened here as per receommendation of this closed issue.

Steps to reproduce

Deactivate an account

Expected behaviour

Accounts should be unavailable for chat and videocalls

Actual behaviour

Contacts show up as able to chat and for videocalls

Server configuration

Nextcloud version: (see Nextcloud admin page)
12.0.0
Contacts version: (see Nextcloud apps page)
1.5.3
Updated from an older Nextcloud or fresh install:
Updated since owncloud 6, migrated to Nextcloud 10 from owncloud 9

@MorrisJobke
Copy link
Member

cc @georgehrke @ChristophWurst

@MorrisJobke
Copy link
Member

cc @nextcloud/spreed

@MorrisJobke
Copy link
Member

Same happens in the contacts dialog: #9824 and the share menu: #6755

@MorrisJobke
Copy link
Member

@danxuliu @skjnldsv Mind to have a look at this?

@MorrisJobke MorrisJobke added this to the Nextcloud 14 milestone Jun 11, 2018
@ChristophWurst ChristophWurst removed their assignment Jun 26, 2018
@nextcloud-bot nextcloud-bot added the stale Ticket or PR with no recent activity label Jul 27, 2018
@tcitworld
Copy link
Member

tcitworld commented Aug 4, 2018

There's two issues :

  • Remove user card from system address book when it's updated and deactivated (easy) - fixes the contact dialog
  • Ignore deactivated users when listing them (for sharees and talk app). It's harder because it makes sql queries harder, and has to be implemented for all backends. In fact it's better to handle it afterwards since the number of deactivated users stays low anyhow.

I'll try to push something soon.

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

Successfully merging a pull request may close this issue.

5 participants