Grader updates - #1329
Conversation
There was a problem hiding this comment.
Pull request overview
Updates the grader experience by improving bulk question selection for regrades and ensuring LTI 1.3 grade passback is triggered when an instructor manually overrides an assignment total.
Changes:
- Add a “Select all” button on the grader questions page to quickly select all questions.
- Trigger an LTI 1.3 score update attempt when setting a manual assignment total override.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| bases/rsptx/assignment_server_api/routers/grader.py | Sends an LTI 1.3 score update attempt after saving a manual total override. |
| bases/rsptx/assignment_server_api/assignment_builder/src/components/routes/Grader/pages/GraderQuestionsPage.tsx | Adds a toolbar “Select all” button to select all questions in the assignment. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| grade = await set_manual_total( | ||
| student.id, assignment.id, course.course_name, payload.score, True | ||
| ) | ||
| await attempt_lti1p3_score_update(student.id, assignment.id, payload.score) |
|
With our LTI workflow, there should not be students in an LTI linked course without LTI1.3 user mappings... I don't think that is going to cause many failed sends. I don't think we want to send the manual score if grades are not released (that would implicitly release the score via LMS gradebook). It will get sent when release state is changed. |
|
Doesn't the column header for the name already allow you to select all the problems? |
|
Ahhh, didn't see that. The default view is Cards which does not have that widget. Maybe Select All button make sense only in that view? |
|
Yeah a button in that view would be good. |
43a9a10 to
6d220b3
Compare
Two updates to the new grader: