Skip to content

Amaresh taking over for Linh – Fixed educator Task Submissions Backend Fixes + Add studentId + Late Detection#1934

Open
linh2020 wants to merge 8 commits into
developmentfrom
linh_educator_taskSubmissions_backend_lateDetection
Open

Amaresh taking over for Linh – Fixed educator Task Submissions Backend Fixes + Add studentId + Late Detection#1934
linh2020 wants to merge 8 commits into
developmentfrom
linh_educator_taskSubmissions_backend_lateDetection

Conversation

@linh2020

@linh2020 linh2020 commented Dec 3, 2025

Copy link
Copy Markdown
Contributor

Description

image

This PR fixes the Educator Task Submissions backend flow for #1934 and adds missing support requested in review:

  • proper studentId filtering support from query params
  • late submission detection/tagging logic
  • stabilized response contract for educator submissions page

Related PRS (if any):

This backend PR is related to the frontend PR#4486

Main changes explained:

  • Updated educator submissions endpoint logic to support expected filters:
    • status
    • student_id (mapped to studentId)
    • course_id (mapped to lesson plan/course)
  • Added late detection logic:
    • marks submission as late when completedAt > dueAt
    • marks item as overdue when due date passed and task not submitted
  • Improved status normalization for educator UI filters:
    • submissions → completed submissions
    • pending submissions → assigned/pending set
  • Kept response compatibility by returning required snake_case fields:
    • student_name, task, submission_link, status, submitted_at
  • Also returns camelCase fields used by current frontend rendering.
  • Fixed backend route/controller wiring issues that previously caused route boot failures in integration tests.

How to test:

  • Checkout this backend branch.

  • Start app and login as Educator/Admin user.

  • Navigate to Educator task submissions view (/educator/task-submissions page in UI flow).

  • Verify:

    • submission list loads dynamically
    • filtering by course, status, and student works
    • records show expected fields (student name, task, type, submission date, status)
    • late submissions are tagged correctly
    • pending/overdue behavior is correct
    • review link opens task detail/review page

Screenshots or videos of changes:

Note:

Include the information the reviewers need to know.

@linh2020 linh2020 force-pushed the linh_educator_taskSubmissions_backend_lateDetection branch from ea4a010 to 5440c6c Compare March 4, 2026 06:49
@sonarqubecloud

sonarqubecloud Bot commented Mar 4, 2026

Copy link
Copy Markdown

@sonarqubecloud

Copy link
Copy Markdown

@one-community one-community added the High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible label Jun 6, 2026
@one-community one-community changed the title Linh – Fixed educator Task Submissions Backend Fixes + Add studentId + Late Detection Amaresh taking over for Linh – Fixed educator Task Submissions Backend Fixes + Add studentId + Late Detection Jun 10, 2026
@sonarqubecloud

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants