devilry / devilry-django

Devilry project main repository
http://devilry.org
BSD 3-Clause "New" or "Revised" License
51 stars 24 forks source link

Adding 1000+ students to examiner fails #1151

Open torgeirl opened 4 years ago

torgeirl commented 4 years ago

Our larges courses have started to exceed 1000 students, an unforeseeable number in the early days of Devilry's development cycle.

Steps to reproduce

Workaround

Solutions a. Give an error when an admin tries to add 1000+ students to an examiner b. Re-write function to only select 997 students when selecting all c. Re-write function to work with 1000+ students.

torgeirl commented 4 years ago

I naively assumed the magic number was 1000 (four digit), but I have been able to reproduce the issue with as «few» as 998 students. 997 students seems to work.