Use getPublicProfile in leaderboard API

[alexanderqchen]

Issue: #29

[alexanderqchen]

I removed the eslint rule to allow for changing function parameters. My reasoning for this was that 1. express requires you to change the properties of the parameters (i.e. req.send()) 2. limit and offset parameters are changed widely in the codebase.

Let me know if you disagree with these rule changes. I'm also open to only allowing property changes in the parameters (which can also be done with eslint).

[HirdayGupta]

Don't turn this off! Instead, for all files where linter complains about this, we can use the following method:

Attendance.getCommitteeLeaderboard = (committee, _offset, _limit) => {
    let offset = _offset;
    let limit = _limit;
}

[HirdayGupta]

@nkansal96 thoughts on this? ^^

[nkansal96]

I think let's go with allowing property changes in the request parameters. Normally, what @HirdayGupta suggested is the way to go, but given that we're also dealing with express it would be better to also allow property changes.