Feature/student work page upgrade (KHO-339)

[ntietje1]

Summary

• Moves a monstrous amount of code into 2 react query hooks
• Adds shared caching between student works list and student work individual queries (blazingly fast :o)

Checklist

• I have performed a self-review of my code
• I have reached out to another developer to review my code
• New and existing unit tests pass locally with my changes
• All Lint and CI checks are passing

[Copilot]

Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (3)

frontend/src/pages/Assignments/StudentSubmission/index.tsx:42

• Calling ErrorToast directly during render may trigger duplicate alerts on re-renders. Consider wrapping this call in a useEffect hook that watches submissionError to trigger the toast only once per error occurrence.

if (submissionError) { ErrorToast("Error loading submission data", "submissionError"); }

frontend/src/pages/Assignments/StudentSubmission/index.tsx:46

• Directly calling ErrorToast in the render path may result in repeated notifications. Moving this logic into a useEffect hook triggered by analyticsError will help control side effects more reliably.

if (analyticsError) { ErrorToast("Error loading analytics data", "analyticsError"); }

frontend/src/hooks/useStudentWorks.ts:127

• Using Date objects as keys in a Map can lead to issues because different Date instances with the same time are not considered equal. Consider using a consistent key format (e.g., an ISO string) for reliable date comparisons.

if (!commitsPerDay.has(currentDate)) { commitsPerDay.set(new Date(currentDate), 0); }