CSC/ECE 517 Fall 2018 E1870 Warn of deadlines and enforce them: Difference between revisions
No edit summary |
No edit summary |
||
| Line 10: | Line 10: | ||
[[File:Afterr1.png]] | [[File:Afterr1.png]] | ||
==2. Functionality Implementation== | ==2. Functionality Implementation== | ||
Currently, when students didn't request enough reviews(e.g. Each student is required to do at least 2 reviews, but s/he only request one in the first round), they can still request more reviews in the second round, but will lose points due to lack of first round reviews. Now, we want to add the | ===Explanation=== | ||
Currently, when students didn't request enough reviews(e.g. Each student is required to do at least 2 reviews, but s/he only request one in the first round), they can still request more reviews in the second round, but will lose points due to lack of first round reviews. Now, we want to add a functionality to split the case. Below is the flow chart that explains the improvement. | |||
[[File:Diagram.png]] | [[File:Diagram.png]] | ||
===Code Level Design=== | ===Code Level Design=== | ||
=Test Plan= | =Test Plan= | ||
==Test checkbox(Instructor side)== | |||
1. Log in as instructor | 1. Log in as instructor | ||
| Line 31: | Line 27: | ||
5. See the checkbox “Allow student to join reviews late” | 5. See the checkbox “Allow student to join reviews late” | ||
==Test task list(Student side)== | |||
===Case 1: Check the checkbox=== | |||
1. Log in as student | 1. Log in as student | ||
| Line 43: | Line 39: | ||
5. Student cannot do the second review without the first one | 5. Student cannot do the second review without the first one | ||
===Case 2: Uncheck the checkbox=== | |||
1. Log in as student | 1. Log in as student | ||
Revision as of 04:15, 21 November 2018
Introduction
Expertiza supports multi-round peer reviews. Currently, in this situation, student who did not do the previous round peer reviews can still do the subsequent round peer reviews, but will lose about 50 points according to the deduction policy. Therefore, it will be fairer to provide instructor an option to decide whether a student can do the subsequent round peer reviews when s/he did not do the previous round peer reviews.
Design Plan
1. UI Improvement
In assignment#edit page “Review strategy” tab, add a checkbox (e.g., “Allow student to join reviews late”*) to allow instructor to decide whether students can do the second round peer reviews without the first round peer reviews; by default this box is unchecked and it only appears when there are multi-round peer reviews.
- Before
- After
2. Functionality Implementation
Explanation
Currently, when students didn't request enough reviews(e.g. Each student is required to do at least 2 reviews, but s/he only request one in the first round), they can still request more reviews in the second round, but will lose points due to lack of first round reviews. Now, we want to add a functionality to split the case. Below is the flow chart that explains the improvement.
Code Level Design
Test Plan
Test checkbox(Instructor side)
1. Log in as instructor
2. Navigate to “Manage…”,then “Assignments”
3. Under “Actions”, choose “Edit”
4. Navigate to “Review Strategy”
5. See the checkbox “Allow student to join reviews late”
Test task list(Student side)
Case 1: Check the checkbox
1. Log in as student
2. Navigate to “Assignments”
3. Choose an assignment
4. Navigate to “Others’ work”
5. Student cannot do the second review without the first one
Case 2: Uncheck the checkbox
1. Log in as student
2. Navigate to “Assignments”
3. Choose an assignment
4. Navigate to “Others’ work”
5. Student can do the second review without the first one