Feature: Option to retain old grading data

[pranavrao145]

Proposed Changes

Closes #5212.

Type of Change

(Write an X or a brief description next to the type or types that best describe your changes.)

Type	Applies?
🚨 Breaking change (fix or feature that would cause existing functionality to change)
✨ New feature (non-breaking change that adds functionality)	X
🐛 Bug fix (non-breaking change that fixes an issue)
🎨 User interface change (change to user interface; provide screenshots)
♻️ Refactoring (internal change to codebase, without changing functionality)
🚦 Test update (change that only adds or modifies tests)
📦 Dependency update (change that updates a dependency)
🔧 Internal (change that only affects developers or continuous integration)

Checklist

(Complete each of the following items for your pull request. Indicate that you have completed an item by changing the [ ] into a [x] in the raw text, or by clicking on the checkbox in the rendered description on GitHub.)

Before opening your pull request:

• I have performed a self-review of my changes.
  • Check that all changed files included in this pull request are intentional changes.
  • Check that all changes are relevant to the purpose of this pull request, as described above.
• I have added tests for my changes, if applicable.
  • This is required for all bug fixes and new features.
• I have updated the project documentation, if applicable.
  • Associated PR: Add info about option to retain existing grading data on recollected submissions Wiki#221

After opening your pull request:

• I have updated the project Changelog (this is required for all changes).
• I have verified that the pre-commit.ci checks have passed.
• I have verified that the CI tests have passed.
• I have reviewed the test coverage changes reported by Coveralls.
• I have requested a review from a project maintainer.

Questions and Comments

Within inline comments.

[coveralls]

Pull Request Test Coverage Report for Build 11958705999

Details

• 236 of 237 (99.58%) changed or added relevant lines in 13 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage decreased (-0.04%) to 91.657%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
app/jobs/submissions_job.rb	23	24	95.83%

Totals
Change from base Build 11955265242:	-0.04%
Covered Lines:	41062
Relevant Lines:	44117

---

💛 - Coveralls

[david-yz-liu]

Hi @pranavrao145, great work! I just did a quick review of the back-end implementation and left some comments. Please work on those, and then I'll do another review in a few days.

[pranavrao145]

I'm just going to convert this back to a draft while I work on addressing comments just so it doesn't run tests on intermediate work, and I'll make it ready for review again once I'm finished!

EDIT: back to ready now, but Jest tests still wrongly failing

[david-yz-liu]

@pranavrao145 great job getting this completed! 🎉