-
Notifications
You must be signed in to change notification settings - Fork 17
Description
Background / Purpose
Within the Code Reviewer Dashboard, currently, we list assignments in the following statuses:
- In Review (for the currently logged-in code reviewer)
- Awaiting Review
- Needs Update (for the currently logged-in code reviewer)
This causes a problem for the instructors (ROLE_INSTRUCTOR) as the instructors are sometimes not the ones performing the code reviews, and therefore they cannot see the "In Review" and "Needs Update" assignments that are currently assigned to other code reviewers (ROLE_CODE_REVIEWER)
Suggested Solution
In the Code Reviewer Dashboard (/src/CodeReviewerDashboard/index.js), when logged in as an instructor (ROLE_INSTRUCTOR) and ONLY when logged in as an instructor, we should display (almost) all assignments without filtering by the code reviewer's user_id field.
Inside AssignmentService there is a findByUser method that currently behaves differently if you are a code reviewer or a student. You'll need to add another behavior that returns all assignments if the logged-in user is an instructor.