Skip to content

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

Draft
kbuffardi wants to merge 1 commit into
mainfrom
feature/module-resource-ordering
Draft

feat(create): persist ordered module resources#183
kbuffardi wants to merge 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’s resource multiselect with an additive, sortable, removable list
  • reject duplicate resource IDs at validation and cover that boundary with unit tests

Closes #182

Verification

  • 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

Test limitation

The validations library has no configured Nx test target. The Cypress scenario was added, but local browser execution cannot reach the protected Create route without the API/auth service, so it times out before the form mounts.

Security note

npm audit --omit=dev --audit-level=high reports pre-existing 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
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