Skip to content

[Feature:RainbowGrades] Add warning for Grade Summary last run#12206

Merged
bmcutler merged 4 commits intomainfrom
add-grade-last-ran-warning
Nov 14, 2025
Merged

[Feature:RainbowGrades] Add warning for Grade Summary last run#12206
bmcutler merged 4 commits intomainfrom
add-grade-last-ran-warning

Conversation

@Rkoester47
Copy link
Contributor

Why is this Change Important & Necessary?

Currently, there is no indication that it has been a long time since the grade summaries were generated. If Nightly Builds are not enabled, this can cause problems. So, to compliment PR#12139 which added a warning for Nightly Builds, this PR will let give instructors an indication that potentially too much time as passed.

What is the New Behavior?

Now, if it has been more than 7 days since Grade Summaries have been generated, a warning next to "Last Ran:..." will be displayed with the number of days since last run.

(Firefox - Light Mode)
LightMode

(Firefox - Dark Mode)
DarkMode

What steps should a reviewer take to reproduce or test the bug or new feature?

  1. Go to Sample (or any test courses that have not run grade summaries in more than 7 days)
  2. Observe the warning text underneath the Generate Grade Summaries button.
  3. Go to a course that has run Generate Grade Summaries in the last week ( or generate it yourself)
  4. Observe that no warning displays.

Other information

This is not a breaking change.

@codecov
Copy link

codecov bot commented Nov 10, 2025

Codecov Report

❌ Patch coverage is 0% with 12 lines in your changes missing coverage. Please review.
✅ Project coverage is 21.71%. Comparing base (77b417f) to head (e1970cd).

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##               main   #12206      +/-   ##
============================================
- Coverage     21.72%   21.71%   -0.01%     
- Complexity     9609     9611       +2     
============================================
  Files           268      268              
  Lines         36056    36068      +12     
  Branches        475      475              
============================================
  Hits           7832     7832              
- Misses        27753    27765      +12     
  Partials        471      471              
Flag Coverage Δ
autograder 21.39% <ø> (ø)
js 2.07% <ø> (ø)
migrator 100.00% <ø> (ø)
php 20.71% <0.00%> (-0.01%) ⬇️
python_submitty_utils 80.08% <ø> (ø)
submitty_daemon_jobs 90.72% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Contributor

@JManion32 JManion32 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code looks good and new functionality works well. This is a necessary quality of life improvement for the grades configuration page.

For anyone who hasn't generated grade summaries on this instance of their VM, you can change your server time with the following in order to view the warning message:

Disable the time from automatically synching:
sudo systemctl stop vboxadd-service

Change the date / time:
sudo date -s "2025-12-07 10:00:00", where the date can be whatever you want as long as it is 7 days from today.

When you are done, reenable:
sudo systemctl start vboxadd-service

@github-project-automation github-project-automation bot moved this from Seeking Reviewer to Awaiting Maintainer Review in Submitty Development Nov 11, 2025
@Rkoester47 Rkoester47 moved this from Awaiting Maintainer Review to Ready to Merge in Submitty Development Nov 13, 2025
@bmcutler bmcutler merged commit 978a5c3 into main Nov 14, 2025
25 checks passed
@bmcutler bmcutler deleted the add-grade-last-ran-warning branch November 14, 2025 00:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants