CSC/ECE 517 Spring 2019/E1927 Let Course Staff as Well as Students Do Reviews: Difference between revisions
Line 3: | Line 3: | ||
='''Project Tasks'''= | ='''Project Tasks'''= | ||
==Task 1:== | ==Task 1:== | ||
'''Problem:''' | '''Problem:''' | ||
Each submission on the submissions page should link to "perform review" instead of "assign grade" iff [the instructor/ta is a participant of the assignment and the last due date for the assignment has not passed] | |||
'''Solution:''' | '''Solution:''' | ||
list_submissions_html.erb should be modified to change the link as noted above. TBD if controller changes in assignment_controler are needed to get due date information and participant information to the view. | |||
==Task 2:== | ==Task 2:== | ||
'''Problem:''' | '''Problem:''' | ||
Review page should allow instructor/TA to submit review if they are a participant of the assignment and the last due date hasn't passed | |||
'''Solution:''' | '''Solution:''' | ||
Check response.html.erb to make sure there aren't any restrictions preventing instructor/ta from submitting reviews if they meet the criteria above. Existing logic may assume all reviewers are student. TBD if any modifications to response_controller is necessary to enable this functionality. | |||
==Task 3:== | ==Task 3:== | ||
'''Problem:''' | '''Problem:''' | ||
An instructor icon should be used to indicate an instructor review and this icon should be seen on the | An instructor icon should be used to indicate an instructor review and this icon should be seen on the review page read by the student. | ||
Revision as of 05:31, 11 April 2019
Problem Summary
In Expertiza, students are able to perform reviews on other students work by filling out a review form on Expertiza. Instructors and TAs for the course should be able to submit a review in the same manner as a student. Only instructors or TAs who are participants in the assignment should be allowed to leave reviews. However, the instructor and TA reviews should be distinguished from other student reviews so the student knows the review was written by the instructor.
Project Tasks
Task 1:
Problem:
Each submission on the submissions page should link to "perform review" instead of "assign grade" iff [the instructor/ta is a participant of the assignment and the last due date for the assignment has not passed]
Solution:
list_submissions_html.erb should be modified to change the link as noted above. TBD if controller changes in assignment_controler are needed to get due date information and participant information to the view.
Task 2:
Problem:
Review page should allow instructor/TA to submit review if they are a participant of the assignment and the last due date hasn't passed
Solution:
Check response.html.erb to make sure there aren't any restrictions preventing instructor/ta from submitting reviews if they meet the criteria above. Existing logic may assume all reviewers are student. TBD if any modifications to response_controller is necessary to enable this functionality.
Task 3:
Problem:
An instructor icon should be used to indicate an instructor review and this icon should be seen on the review page read by the student.
Solution:
Modify the add-ta-128.png icon file to look more like an instructor and add it to the view.
Task 4:
Problem:
Refactor: A "course_staff" column was added to the "response_maps" table. Is it really necessary to have a column named as "course_staff" for reviews?
Solution: Remove the column name from the response maps, if it exists.
Find another way of indicating the review is an instructor review.
Task 5: (OPTIONAL)
Problem:
Similar to the calibration report from the Design Exercise assignment, create a view that would allow the student reviewers differ from the reviews completed by the course staff.
Solution:
Review the code from the Design Exercise calibration assignment. Discuss ideas with the professor (as advised from previous mentor meetings) to form a plan on how to address this task.
Test Plan
Write testing methodology (logic) here.
References
1. Project Document: https://docs.google.com/document/d/1Ozw2Bj2u_LyeKUdDqAyQtz1stEIKrxkmRnwmNyUgmzw/edit#heading=h.bumi2mscmmgf
2. Github Repo: https://github.com/mblewallncsu/expertiza
3. Previous Work, Project E1824: http://wiki.expertiza.ncsu.edu/index.php/CSC/ECE_517_Spring_2018-_Project_E1824:_Let_course_staff_as_well_as_students_do_reviews
4. Previous Work, Project E1855: http://wiki.expertiza.ncsu.edu/index.php/CSC/ECE_517_Fall_2018/E1855_let_course_staff_as_well_as_students_do_reviews
Team Members
1. Jimmy Boykin
2. Srikara Krishna
3. Michael Lewallen