Add ability to search through applicants#186
Conversation
petschekr
added
enhancement
|
Hey y'all! A deployment of this PR can be found here: |
There was a problem hiding this comment.
No it's not, I'll fix that
There was a problem hiding this comment.
What's the difference between accepted and not-confirmed? Where does not-confirmed apply to a user (since a single application branch can potentially have multiple confirmation branches)?
There was a problem hiding this comment.
Accepted includes all accepted applicants regardless of whether or not they have confirmed whereas not confirmed only includes accepted applicants who have not confirmed
There was a problem hiding this comment.
@illegalprime Is there a way to just specify "Get all info about the user"? That would clean up this query a lot
There was a problem hiding this comment.
Do we have to worry about escaping here since this is user supplied input?
There was a problem hiding this comment.
Escaping happens on line 54 right above this. Input is normally escaped unless the user ticks the Use Regex box in which case it is directly passed through. I don't see any security implications to doing that because the regular expressions only retrieve data and because this feature is already restricted to admins only.
There was a problem hiding this comment.
Might want to refactor this later if we ever want to add other login providers in a modular way
…e, and form item labels to GraphQL API
petschekr
force-pushed
the
search-applicants
branch
from
869501e to
b11bb20
Compare
petschekr
force-pushed
the
search-applicants
branch
from
25235be to
f93e73b
Compare
3 participants
Probably coming soon:tm:: displaying confirmation data in addition to the application data already present
Closes #183