Skip to content
This repository was archived by the owner on Feb 8, 2018. It is now read-only.

(ignore) Allow admins to search by email #4145

Closed
wants to merge 10 commits into from
Closed

(ignore) Allow admins to search by email #4145

wants to merge 10 commits into from

Conversation

mattbk
Copy link
Contributor

@mattbk mattbk commented Oct 20, 2016

Closes #3382.

If you are an admin, search will return users whose email address (in participants) matches the query. If you are not an admin, you won't notice the functionality (or lack of).

One thing I couldn't figure out was how to match partial strings. E.g., searching for alice returns both ~user Alice (on username) and ~user Alice (on email_address [email protected]), searching for al only returns Alice on username. However, for support cases, I don't think this is a huge problem because we have the entire email address to search with (as long as they contact us with the email address in the database).

@mattbk
Copy link
Contributor Author

mattbk commented Oct 20, 2016

Looks like I had some changes that I didn't revert first.

@mattbk
Copy link
Contributor Author

mattbk commented Oct 20, 2016

Closing and will redo.

@mattbk mattbk closed this Oct 20, 2016
@mattbk mattbk deleted the search-emails branch October 20, 2016 03:31
mattbk added a commit that referenced this pull request Oct 20, 2016
@mattbk mattbk changed the title Allow admins to search by email (ignore) Allow admins to search by email Oct 20, 2016
@chadwhitacre chadwhitacre mentioned this pull request Oct 21, 2016
7 tasks
chadwhitacre pushed a commit that referenced this pull request Oct 22, 2016
chadwhitacre pushed a commit that referenced this pull request Oct 22, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant