coderabbitai[bot]
commented
4 months ago Walkthrough
## Walkthrough The recent updates focus on enhancing the flexibility and intuitiveness of grade release mechanisms within a learning management system. Notably, the changes introduce the ability to release or unrelease individual student grades and refine the logic behind grade visibility and deadline adherence. This is achieved through modifications across controllers, models, views, and configurations, streamlining the process for instructors and TAs and addressing specific user needs for more granular control over score dissemination. ## Changes | File Path | Change Summary | |-----------|----------------| | `app/controllers/submissions_controller.rb` | Added `release_student_grade` and `unrelease_student_grade` actions with updated `before_action` callbacks. | | `app/helpers/submission_helper.rb` | Added an empty line. | | `app/models/submission.rb` | Added `problems_released?` method; refactored `grades_released?`. | | `app/views/.../viewGradesheet.html.erb`, `app/views/submissions/view.html.erb`, `app/views/submissions/_grades.html.erb` | Updated to support conditional grade release display and logic. | | `app/assets/javascripts/annotations.js`, `app/views/submissions/view.js.erb` | Enhanced grade release button functionality and rendering. | | `app/views/assessments/_handin_form.html.erb` | Refactored late submission logic and form submission adjustments. | | `config/routes.rb` | Added routes for `release_student_grade` and `unrelease_student_grade`. | | `db/schema.rb` | Database schema adjustments including type changes and added limits. | ## Assessment against linked issues | Objective | Addressed | Explanation | |-----------|-----------|-------------| | Allow release of individual student scores (#2063) | ✅ | | | Simplify the grade release process and address grading deadline confusion (#1717) | ❓ | The changes seem to support more granular control over grade releases, which could simplify the process, but it's unclear if the grading deadline confusion is directly addressed. | ## Related issues - **autolab/Autolab#1717**: This issue discusses refining the grading deadline to make it less confusing and more intuitive. The changes in this PR, particularly those enhancing the control over grade release, could indirectly address concerns raised in this issue by providing more flexibility in how and when grades are released, potentially making the grading deadline less of a bottleneck.Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
damianhxy
Description
Screenshots:
Motivation and Context
-Currently, scores can only be released all at once, or through the individual problems on a gradesheet
How Has This Been Tested?
Types of changes
Checklist:
overcommit --install && overcommit --signto use pre-commit hook for linting