Skip to content

feat(create): persist ordered module resources - #183

Merged
kbuffardi merged 1 commit into
mainfrom
feature/module-resource-ordering
Sep 14, 2026
Merged

feat(create): persist ordered module resources#183
kbuffardi merged 1 commit into
mainfrom
feature/module-resource-ordering

Conversation

@kbuffardi

@kbuffardi kbuffardi commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

Summary

  • persist zero-based Module resource ordering with an explicit join model and migration
  • return ordered resources through Module, course, instructor, student, and Demo-attempt read paths
  • order learner Demo exercises by the existing persisted association order
  • replace Module resource multiselect with an additive, sortable, removable list
  • reject duplicate resource IDs at validation
  • correct Demo drag direction after the editor update from fix demo exercise ordering #174 #179

Closes #182

Verification

  • API regression tests cover Module resource create, update, and retrieval order
  • API route regression test covers learner Demo exercise order
  • unit test covers immutable drag reordering and direction
  • Cypress scenarios cover Module reorder, Module removal, and Demo save/reopen ordering
  • npx tsc -p api/tsconfig.app.json --noEmit
  • npx tsc -p client/tsconfig.app.json --noEmit
  • npx nx build api --configuration=production
  • npx nx build client --configuration=production

Security note

npm audit --omit=dev --audit-level=high previously reported repository-wide vulnerabilities, including a critical Sequelize advisory; no dependency changes are included in this PR.

Store zero-based ModuleResources ordering, return it across module and course reads, and make resource selection sortable and removable. Order learner Demo attempts by the existing exercise order.\n\nCloses #182
@kbuffardi
kbuffardi marked this pull request as ready for review September 14, 2026 06:41
@kbuffardi
kbuffardi merged commit 9ab8b75 into main Sep 14, 2026
2 checks passed
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.

fix(create): persist and display Demo and Module item ordering

1 participant