CSC/ECE 517 Spring 2019/E1927 Let Course Staff as Well as Students Do Reviews: Difference between revisions

From Expertiza_Wiki
Jump to navigation Jump to search
Line 3: Line 3:


='''Project Tasks'''=
='''Project Tasks'''=


==Task 1:==
==Task 1:==
'''Problem:'''
'''Problem:'''


An instructor or TA who is a participant of an assignment, then the instructor should be able to review any team that submitted the assignment.
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:'''
(Find the code that validates the TA is a participant. See the comments in the project description for more information.)
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:'''


If the last due date for the assignment has not passed, then the “Assign grade” link should be changed to “Perform review”, and when clicked on, it should pull up a review page for the team, showing their submitted work at the top, as a review page normally does.
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:'''


(Put the name of the file to be revised here.  Show the current code, the revised code, and your results.)
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 instructor review page read by the student.
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.


Add a TA

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