Skip to content
This repository has been archived by the owner on Apr 17, 2023. It is now read-only.

Use typeahead when adding users to a team #39

Closed
eotchi opened this issue May 6, 2015 · 8 comments
Closed

Use typeahead when adding users to a team #39

eotchi opened this issue May 6, 2015 · 8 comments

Comments

@eotchi
Copy link
Contributor

eotchi commented May 6, 2015

Improve the usability of adding a user to a team.
Typeahead or similar (*) should be introduced so to suggest a list of valid users matching the current input of the user.

(*) https://twitter.github.io/typeahead.js/examples/

@flavio
Copy link
Member

flavio commented Jun 11, 2015

On issue #103 @mssola proposed the usage of chosen.

What do you (@mssola) think about typeahead?

@mssola
Copy link
Collaborator

mssola commented Jun 15, 2015

@flavio I have no preference between any of the proposed solutions. Chosen was the one that @cyntss suggested me. So... I don't know :D

@flavio
Copy link
Member

flavio commented Jun 15, 2015

I'll let you and @cyntss figure out what to use. I have no preference either.

@brycereitano
Copy link
Contributor

I was thinking about his feature, but should a non-admin be able to see a list of users? The type-ahead would expose that.

A solution to this, is a non-admin doesn't get the typeahead where the admins would. Otherwise we just expose a list of users to any of the users.

@flavio
Copy link
Member

flavio commented Oct 5, 2015

Yes that's true, but I don't see that as a big security issue. Do you? Sites like github are doing the same thing; also organizations have already shared addressbooks containing this information...

@brycereitano
Copy link
Contributor

I do not, but I thought I would bring it up.

Would you mind if I pick up this issue?

@flavio
Copy link
Member

flavio commented Oct 5, 2015

Sure, go ahead! :)

@mssola
Copy link
Collaborator

mssola commented Nov 6, 2015

Note that this also should apply to team names in the "Create namespace" form inside of the namespaces page.

jloehel pushed a commit to jloehel/Portus that referenced this issue Nov 9, 2015
jloehel pushed a commit to jloehel/Portus that referenced this issue Nov 9, 2015
jloehel pushed a commit to jloehel/Portus that referenced this issue Nov 11, 2015
jloehel pushed a commit to jloehel/Portus that referenced this issue Nov 11, 2015
jloehel pushed a commit to jloehel/Portus that referenced this issue Nov 11, 2015
jloehel pushed a commit to jloehel/Portus that referenced this issue Nov 11, 2015
jloehel pushed a commit to jloehel/Portus that referenced this issue Nov 11, 2015
jloehel pushed a commit to jloehel/Portus that referenced this issue Nov 12, 2015
jloehel pushed a commit to jloehel/Portus that referenced this issue Nov 12, 2015
jloehel pushed a commit to jloehel/Portus that referenced this issue Nov 12, 2015
jloehel pushed a commit to jloehel/Portus that referenced this issue Nov 12, 2015
jloehel pushed a commit to jloehel/Portus that referenced this issue Nov 12, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants