Skip to content

[Feature:InstructorUI] Add Icon to Editing Peer Components#13031

Merged
bmcutler merged 1 commit into
mainfrom
add-peer-icon-to-edit-component
Jul 15, 2026
Merged

[Feature:InstructorUI] Add Icon to Editing Peer Components#13031
bmcutler merged 1 commit into
mainfrom
add-peer-icon-to-edit-component

Conversation

@mattLif

@mattLif mattLif commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

Why is this Change Important & Necessary?

Adds an icon to peer components on the edit rubric page to further emphasize that they are distinct from regular components.

What is the New Behavior?

Before:
image

After:
image

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

As an instructor, visit the edit rubric page any gradeable with peer components and notice icon.

Automated Testing & Documentation

No automated testing added

Other information

Not a breaking change
No migrations
Not a security risk

@codecov

codecov Bot commented Jul 13, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 21.29%. Comparing base (161a27f) to head (fd71e7c).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##               main   #13031   +/-   ##
=========================================
  Coverage     21.29%   21.29%           
  Complexity    10098    10098           
=========================================
  Files           278      278           
  Lines         37794    37794           
  Branches        516      516           
=========================================
  Hits           8050     8050           
  Misses        29241    29241           
  Partials        503      503           
Flag Coverage Δ
autograder 21.16% <ø> (ø)
js 1.98% <ø> (ø)
migrator 100.00% <ø> (ø)
php 20.21% <ø> (ø)
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.

@GarvitKhandelwal31 GarvitKhandelwal31 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Everything lgtm

@github-project-automation github-project-automation Bot moved this from Seeking Reviewer to Awaiting Maintainer Review in Submitty Development Jul 13, 2026

@dandrecollins07-ctrl dandrecollins07-ctrl left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Functionality works as intended, code looks good. Approving.

@GarvitKhandelwal31 GarvitKhandelwal31 moved this from Awaiting Maintainer Review to Ready to Merge in Submitty Development Jul 13, 2026
@bmcutler
bmcutler merged commit bb6d4a7 into main Jul 15, 2026
30 of 31 checks passed
@bmcutler
bmcutler deleted the add-peer-icon-to-edit-component branch July 15, 2026 15:38
@github-project-automation github-project-automation Bot moved this from Ready to Merge to Done in Submitty Development Jul 15, 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.

4 participants