autolab / Autolab

Course management service that enables auto-graded programming assignments.
http://www.autolabproject.com/
Apache License 2.0
752 stars 213 forks source link

autolab should validate the embedded quiz file #892

Open cg2v opened 6 years ago

cg2v commented 6 years ago

Instructors are trying to upload non-conforming files as embedded quiz data (e.g. pdfs) which triggers a DB exception so they get no feedback. The controller or model should do some basic validation and display a message for uploads that are not appropriate.

Maybe there should also be a checkbox: "I have read and understood the embedded quiz documentation" to enable the upload button.

TheodorJ commented 6 years ago

Agreed. I think adding the validation would be the best way to fix this, since the checkbox probably won't get people to read the documentation (like how nobody reads Terms and Conditions before agreeing). Leaving this issue open.