-
Notifications
You must be signed in to change notification settings - Fork 273
Adds route and controller for missed update user list api #1766
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
@Ajeyakrishna-k can you resolve the merge conflicts please. |
cf06175
to
a71b3a6
Compare
Done! |
); | ||
router.patch("/assign/self", authenticate, invalidateCache({ invalidationKeys: [ALL_TASKS] }), tasks.assignTask); | ||
|
||
router.get("/users/discord", verifyCronJob, getUsersValidator, tasks.getUsersHandler); |
Check failure
Code scanning / CodeQL
Missing rate limiting
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Please make sure of raising another PR where you have defined constants in upper and snake case.
Date: 9 Dec 2023
Developer Name: @Ajeyakrishna-k
Issue Ticket Number:-
Description:
Adds a route and controller function which returns discord Id of users who are active with tasks but have not filled in progress updates on task details page.
Docs:
Is Under Feature Flag
Database changes
Breaking changes (If your feature is breaking/missing something please mention pending tickets)
Is Development Tested?
Add relevant Screenshot below ( e.g test coverage etc. )