Skip to content

[Bugfix:RainbowGrades] Revert reload on build success#13049

Merged
bmcutler merged 1 commit into
mainfrom
fix-grades-config
Jul 16, 2026
Merged

[Bugfix:RainbowGrades] Revert reload on build success#13049
bmcutler merged 1 commit into
mainfrom
fix-grades-config

Conversation

@dagemcn

@dagemcn dagemcn commented Jul 16, 2026

Copy link
Copy Markdown
Member

Why is this Change Important & Necessary?

In PR#13026, I added a reload on build success to clear the manual generation banner if it was present. This had the side effect of changing the message shown by the output of building rainbow grades, which broke the grades configuration spec. The reload would also clear the log button, so this change is better reverted and I can work on a fix for clearing the banner in another PR.

What is the New Behavior?

The page will no longer reload on a successful build of rainbow grades.

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

  1. Checkout the branch and do submitty_install_site
  2. Do a manual generation of rainbow grades
  3. After the banner appears, click build in the UI
  4. The page should no longer reload

Other information

N/A

@github-project-automation github-project-automation Bot moved this to Seeking Reviewer in Submitty Development Jul 16, 2026
@automateprojectmangement automateprojectmangement Bot moved this from Seeking Reviewer to Work in Progress in Submitty Development Jul 16, 2026
@codecov

codecov Bot commented Jul 16, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 21.12%. Comparing base (f002df1) to head (b3d4b82).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##               main   #13049   +/-   ##
=========================================
  Coverage     21.12%   21.12%           
  Complexity    10114    10114           
=========================================
  Files           279      279           
  Lines         38125    38125           
  Branches        578      578           
=========================================
  Hits           8053     8053           
  Misses        29507    29507           
  Partials        565      565           
Flag Coverage Δ
autograder 21.16% <ø> (ø)
js 1.78% <ø> (ø)
migrator 100.00% <ø> (ø)
php 20.20% <ø> (ø)
python_submitty_utils 79.83% <ø> (ø)
submitty_daemon_jobs 91.13% <ø> (ø)
vue 100.00% <ø> (ø)

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.

@dagemcn
dagemcn marked this pull request as ready for review July 16, 2026 14:38
@automateprojectmangement automateprojectmangement Bot moved this from Work in Progress to Seeking Reviewer in Submitty Development Jul 16, 2026
@github-project-automation github-project-automation Bot moved this from Seeking Reviewer to Awaiting Maintainer Review in Submitty Development Jul 16, 2026
@bmcutler
bmcutler merged commit 60617d0 into main Jul 16, 2026
31 checks passed
@bmcutler
bmcutler deleted the fix-grades-config branch July 16, 2026 15:58
@github-project-automation github-project-automation Bot moved this from Awaiting Maintainer Review to Done in Submitty Development Jul 16, 2026
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.

2 participants