feat: getAllApplications

[ojn03]

ℹ️ Issue

Closes #44 #59

📝 Description

Added GET endpoint at route "api/apps/"

If calling user is not a recruiter or admin, return 401 unauthurized. otherwise returns all applications of current cycle

response is an array of GetApplicationResponseDTO:
[
GetApplicationResponseDTO,
GetApplicationResponseDTO,
...
]

this is a bit different from what was specified in the ticket:
{
"applications": GetApplicationResponseDTO[]
}

because I believe it falls more in line with the existing endpoints and type definitions. If necessary I can change it to follow the ticket more exactly.

Comments/Questions:

• I was unsure how to test this with postman. Im not sure how to emulate a recruiter/admin/other role as the calling user through postman. How would I do so?

• Does req.user.status hold the calling users role (admin, Member, recruiter, etc)? I assumed so by looking at other endpoints, but Im not certain

• Is there a reason the fields in the Cycle class were made private? I had to make them public to make this work

• What is the numapps parameter in the application.toGetApplicationResponseDTO() function supposed to represent?

[kimharr24]

Nice work! Added a few comments addressing your TODOs.

1. For testing, you can sign up using your email and sign in, at which point your user information should be stored as a row in the Users table. You can directly modify your status field to be an Admin/Recruiter in pgAdmin for testing this endpoint. You can also add applications to the Applications table directly in pgAdmin.
2. Yes
3. Not sure, generally keeping fields private is good practice - I think @chromium-52 made this design decision
4. The number of times an applicant has applied in the past

[chromium-52]

💯 Just some quick comments but lgtm otherwise!