Skip to content

Feat 323 assignment bugs - #324

Open
karimouf wants to merge 7 commits into
devfrom
feat-323-assignment-bugs
Open

Feat 323 assignment bugs#324
karimouf wants to merge 7 commits into
devfrom
feat-323-assignment-bugs

Conversation

@karimouf

@karimouf karimouf commented Aug 4, 2026

Copy link
Copy Markdown
Collaborator

Overview

Three bug fixes around assignment revisions and assignment copying:

  • The "max revisions" slider's unlimited sentinel collided with the newly-allowed "0 revisions" option
  • The max-revisions error message exposed a confusing internal count
  • Copying an assignment didn't record which assignment it was copied from

Improvements

  • Clarified the Maximum Revisions slider help text: N revisions means the original submission plus N resubmissions (e.g. 1 revision = 2 submissions total)
  • Simplified the "maximum revisions reached" error message shown to users, dropping the internal (chainDepth/maxRevisions) count that could read as inconsistent with the revision terminology
  • Assignment cards now show a "submissions left" badge and a due-date badge (moved into the card header, next to the upload button) instead of no indication at all
  • Due-date badge turns orange within 24 hours of closing and red (with an "Overdue by ..." message) once the due date has passed, instead of always showing a plain, non-directional time difference
  • "Submissions left" badge turns green once a user has 0 submissions remaining
  • Users with the upload-for-others right see the due date for everyone but no longer see a personal "submissions left" count, since it isn't meaningful for them

Bug fixes

  • Fixed: the Maximum Revisions slider was fixed; the number described for each revision adding to your first submission we now can set the maximum revisions to zero, which means you only can upload one original submission
  • Fixed: unlimited revisions is now set to -1 instead of a specific number
  • Fixed: copying an assignment not recording its origin — parentAssignmentId is now set to the source assignment's ID when using "Copy Assignment", so copies stay traceable to their origin

Comment thread backend/webserver/sockets/document.js Outdated
@karimouf
karimouf requested a review from dennis-zyska August 11, 2026 11:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants