-
Notifications
You must be signed in to change notification settings - Fork 1k
Description
Describe the bug
Comparing these two sample configs:
more_autograding_examples/pdf_exam/config/config.json more_autograding_examples/upload_only_100mb/config/config.json
Other than the max submission size being different, the only real difference is these two config settings:
// don't show students the version # or the submitted files box
// (they can still view and download pdf files from the active
// submission when manual TA grading is released)
"hide_submitted_files" : true,
"hide_version_and_test_details" : true
Which appear to no longer work as originally designed, as described in the comment.
The scanned upload .pdf is never made visible to the students, even after grades are released.
I believe it did work -- students were able to viewed/download the scanned pdf that was made available to them.
I'm not sure if it's part of the open bug from the dates (release date/due date) @cjreed121 is working on
Or unrelated and its been broken for a while. I don't think it's just the dates, setting all those values correctly as a workaround, its still not visible to students. Swapping the pdf_exam config, replacing it with the upload_only config (removing those options) fixes the problem (students can view/download their pdf).
Let's fix it, and make some automated tests.
(tagging a few of you on this)