Skip to content

[Feature:RainbowGrades] Registration type in grade summaries#8026

Merged
bmcutler merged 6 commits intomainfrom
reg-type-7474
Jun 21, 2022
Merged

[Feature:RainbowGrades] Registration type in grade summaries#8026
bmcutler merged 6 commits intomainfrom
reg-type-7474

Conversation

@tkoz0
Copy link
Copy Markdown
Contributor

@tkoz0 tkoz0 commented Jun 11, 2022

Please check if the PR fulfills these requirements:

  • Tests for the changes have been added/updated (if possible)
  • Documentation has been updated/added if relevant

What is the current behavior?

The grade summaries for rainbow grades do not include registration type.
Closes #7474

What is the new behavior?

The JSON files used by rainbow grades now have the registration type included.

Other information?

I tested by setting various registration types in the sample course. Then after generating the grade summaries, checking the JSON files to see that the correct "registration_type" property is set.

@codecov
Copy link
Copy Markdown

codecov bot commented Jun 11, 2022

Codecov Report

Merging #8026 (b75c61b) into main (fcd5438) will decrease coverage by 0.00%.
The diff coverage is 50.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##               main    #8026      +/-   ##
============================================
- Coverage     22.20%   22.19%   -0.01%     
  Complexity     7581     7581              
============================================
  Files           203      203              
  Lines         24775    24775              
  Branches         60       60              
============================================
- Hits           5501     5500       -1     
- Misses        19216    19217       +1     
  Partials         58       58              
Flag Coverage Δ
autograder 20.22% <ø> (ø)
js 31.34% <ø> (ø)
migrator 99.20% <ø> (ø)
php 20.32% <50.00%> (-0.01%) ⬇️
python_submitty_utils 71.65% <ø> (ø)

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

@bmcutler bmcutler merged commit 1c3d059 into main Jun 21, 2022
@bmcutler bmcutler deleted the reg-type-7474 branch June 21, 2022 15:49
@bmcutler bmcutler mentioned this pull request Aug 22, 2022
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

column for registration type

2 participants