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

Possible regression: when (re)labeling a conversation, users not getting styled for conflicts #62

Closed
douglasrapp opened this issue Aug 19, 2015 · 4 comments
Assignees

Comments

@douglasrapp
Copy link

When labeling or relabeling a direct message or private group, potential/selected users are not getting highlighted when there is a security label conflict under 2 situations:

  1. When typing in the auto-complete box to add a user, the entries in the drop-down list of users to select are never styled, even when some of those users do not have access to current labels.

  2. In a private group, when a user is selected and their corresponding "chiclet" entry appears underneath the input box, that "chiclet" is never styled, even if security labels are selected that cause a conflict.

In both cases, the relevant security labels still get styled, and the pertinent warnings still get displayed as expected.

I believe this is a regression (pretty sure this was working earlier), and possibly caused by the change from displaying usernames to displaying full names. It is possible that the security label code is operating on usernames, while the drop-down/chiclet fields are only including full name data, and so they never match.

@douglasrapp douglasrapp added this to the Product Backlog milestone Aug 19, 2015
@bbrockman
Copy link

That was working before.

@douglasrapp
Copy link
Author

Created branch bugfix/chat-locker-62

@rwakida
Copy link

rwakida commented Aug 22, 2015

👍

@douglasrapp
Copy link
Author

Merged branch (pull request #66).

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

No branches or pull requests

3 participants