Skip to content

Feat 120 assigment dashboard#142

Open
karimouf wants to merge 71 commits into
devfrom
feat-120-assigment_dashboard
Open

Feat 120 assigment dashboard#142
karimouf wants to merge 71 commits into
devfrom
feat-120-assigment_dashboard

Conversation

@karimouf
Copy link
Copy Markdown
Collaborator

@karimouf karimouf commented Mar 31, 2026

Main Description

Implemented an Assignment Dashboard flow that shifts submission handling from admin-only/manual import to assignment-based self-service uploads, enabling both admins and users to manage submissions and revision workflows.


User Features

  • Create assignments and assign to roles
  • Edit / Copy / Delete / Close assignments
  • View submissions per assignment
  • Submit multiple revisions
  • Replace/Delete submissions (if not used in a study)
  • View and manage assignment-bound submissions
  • Submission metadata support (description)
  • Study usage tracking via studyUsageCount
  • Assignment-specific views for submissions
  • Max revisions slider (including unlimited option)

System Features

  • Assignment Data Model

    • title, description, public
    • studyId, workflowId, userId
    • start, end, closed
    • validationConfigurationId
    • assignedRoleIds
    • parentAssignmentId, previousSubmissionAssignmentId
    • allowReUpload, maxRevisions
    • deleted, deletedAt, createdAt, updatedAt
  • Submission Model Updates

    • name, description (metadata)
    • assignmentId (linkage)
  • Access Control

    • Own + public visibility
    • Admin: full access
    • Owner-only actions (Edit/Delete/Share)
  • Submission Pipeline

    • Assignment-based uploads
    • Revision chain enforcement
  • Validation & Rules

    • Validation before persistence
    • Upload only when open (admin override)
    • Re-upload controlled via configuration
  • Revision Control

    • Max revision limits
    • Standard naming (revision X)
  • Study Protection

    • Prevents unsafe replace/delete using studyUsageCount

Known Limitations

  • User assignment view still missing some features (pending alignment/decision)

karimouf and others added 30 commits March 17, 2026 14:53
Copy link
Copy Markdown
Collaborator

@dennis-zyska dennis-zyska left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, nice and fast implementation, I made some comments for improvements. Note that I didn't run the branch yet, so the feedback is only on the code. There are also some merge conflicts.

Comment thread backend/db/migrations/20260316122741-create-assignment.js Outdated
Comment thread backend/db/migrations/20260316122741-create-assignment.js Outdated
Comment thread backend/db/migrations/20260316122741-create-assignment.js Outdated
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can't we add this in the assignment creation migration? As long it is not merged to dev, I think it would be fine to do so. I'm also thinking, maybe it directly make sense here to adapt it also with an additional table instead of an array. Arrays in db tables are always a bad idea (also see https://en.wikipedia.org/wiki/Database_normalization)

Comment thread backend/db/models/assignment.js Outdated
Comment thread frontend/src/components/dashboard/assignments/AdminAssignmentsView.vue Outdated
Comment thread frontend/src/components/dashboard/assignments/AdminAssignmentsView.vue Outdated
Comment thread frontend/src/components/dashboard/Assignments.vue Outdated
Comment thread frontend/src/components/dashboard/Submissions.vue
@karimouf karimouf requested a review from dennis-zyska April 26, 2026 10:20
Comment thread backend/db/migrations/20260418000000-create-assignment_share.js
Comment thread backend/db/models/document.js Outdated
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

both

@karimouf karimouf requested a review from dennis-zyska May 4, 2026 13:44
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.

3 participants