Add ability to search through applicants#186
Conversation
|
Hey y'all! A deployment of this PR can be found here: |
api.graphql
Outdated
There was a problem hiding this comment.
No it's not, I'll fix that
client/admin.html
Outdated
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
client/js/admin.ts
Outdated
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
server/routes/api/graphql.ts
Outdated
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.
server/routes/api/graphql.ts
Outdated
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
869501e to
b11bb20
Compare
25235be to
f93e73b
Compare
Probably coming soon:tm:: displaying confirmation data in addition to the application data already present
Closes #183