devilry / devilry-django

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

Remove old delivery models #1304

Closed espenak closed 2 months ago

espenak commented 3 months ago

We have some old models that was previously used to store deliveries. These are no longer in use:

Need to include in migration guide

Testing

All student views:

All examiner views:

All admin views:

espenak commented 3 months ago

@Levijatan this is ready for testing in the feature/remove-old-delivery-models/main branch.

Levijatan commented 3 months ago

Currently it fails to download delivered files because of Failed to generate zip archive from feedbackset 25 Traceback (most recent call last): File "/Users/asmundrosendahl/work/devilry-django/devilry/devilry_group/tasks.py", line 23, in execute print_memory_usage('Start of RQ task') File "/Users/asmundrosendahl/work/devilry-django/devilry/utils/memorydebug.py", line 33, in print_memory_usage import humanize ModuleNotFoundError: No module named 'humanize'

Levijatan commented 3 months ago

Student dashboard says "(4st forsøk)" for the fourth try in Norwegian

espenak commented 3 months ago

Currently it fails to download delivered files because of Failed to generate zip archive from feedbackset 25 Traceback (most recent call last): File "/Users/asmundrosendahl/work/devilry-django/devilry/devilry_group/tasks.py", line 23, in execute print_memory_usage('Start of RQ task') File "/Users/asmundrosendahl/work/devilry-django/devilry/utils/memorydebug.py", line 33, in print_memory_usage import humanize ModuleNotFoundError: No module named 'humanize'

Just change

DEVILRY_MEMORY_DEBUG_ENABLED = True

to False in develop settings to avoid that @Levijatan. It should be False, but I must have comitted it as True.

Levijatan commented 3 months ago

There are some weirdness around tries but this is hopefully fixed when the fix for #1301 is merged inn. On the fourth try it said "4st forsøk" but on the fifth it says "fjerde forsøk", but the fourth try was given before the third try had been given feedback. The fifth was given through bulk. The fix for #1301 makes it so if there is no feedback given the try deadline is moved instead of creating a new try.

Levijatan commented 3 months ago

Getting "Automatisk anonym ID mangler" when "Half anonymization" is enabled. Except when the student and examiner is the same user then it just showes the name of the user.

espenak commented 2 months ago

Getting "Automatisk anonym ID mangler" when "Half anonymization" is enabled. Except when the student and examiner is the same user then it just showes the name of the user.

I do not think this is an issue, just a result of how the anonymization utils / template tags work. It makes sense that you are not anonymized from yourself, AND it makes absolutely no sense to be your own examiner, especially when anonymized (which is typically an exam). The "Automatisk anonym ID mangler" is probably just because of setup in the dev database.