Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Student should not hold many outstanding reviews without doing any #473

Closed
yangsong8 opened this issue Jan 29, 2015 · 2 comments
Closed

Comments

@yangsong8
Copy link
Contributor

This is in current production. Students can hold many reviews without starting reviewing any. This cause the authors receive many empty or useless reviews. We need to place restrictions to this. The best fix should be that students can “save” and “submit” reviews. Students cannot request more reviews if (s)he hold K outstanding reviews. A student holding k reviews must drop or submit a review before being allowed to choose another artifact to review. If a review is submitted, it cannot be changed till next round. Instructor needs to announce to class that they are only going to get points for up to M (should be on syllabus) reviews. They are welcomed do more reviews but they will not get extra points. (We need to add submit button in addition to save button)

This should be implemented in production first.

@yangsong8
Copy link
Contributor Author

Dr. Gehringer will announce to CSC517 that only 7 reviews will count. In this way there is no hurry for this feature.

@yangsong8
Copy link
Contributor Author

This is what we do in current Rails 4 branch: when students holds 3 reviews without starting them, they cannot request more reviews. There is no "drop" button there, should we have one?

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant