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

admins should be able to search everyone #3121

Closed
chadwhitacre opened this issue Jan 15, 2015 · 4 comments
Closed

admins should be able to search everyone #3121

chadwhitacre opened this issue Jan 15, 2015 · 4 comments
Labels
Ready to Start TeamX

Comments

@chadwhitacre
Copy link
Contributor

is_searchable should only apply to public/user search. Hidden users should still be searchable by admins. (I tried finding someone during review on #3118.)

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.
@Changaco Changaco added TeamX and removed ★★☆ labels Feb 23, 2015
@chadwhitacre chadwhitacre mentioned this issue Dec 19, 2015
Closed
@kzisme
Copy link
Contributor

kzisme commented Dec 22, 2015

Fairly certain this issue relates here

@rohitpaulk
Copy link
Contributor

Yep, all the queries on that page seem to filter participants by is_searchable. We could make those constraints dynamic, and relax them depending on whether the logged in user is an admin or not.

@mattbk mattbk mentioned this issue Nov 13, 2016
Closed
@mattbk
Copy link
Contributor

mattbk commented Nov 13, 2016

PR is #4190, ready for review.

@mattbk mattbk mentioned this issue Nov 13, 2016
Closed
@mattbk
Copy link
Contributor

mattbk commented Jan 1, 2017

This accidentally made it in. Explanation in #4190 (comment).

@mattbk mattbk closed this as completed Jan 1, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
Ready to Start TeamX
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@chadwhitacre @Changaco @kzisme @mattbk @rohitpaulk