Skip to content

[Feature:InstructorUI] Show username on create course page#13025

Merged
bmcutler merged 1 commit into
mainfrom
username-on-create-course
Jul 11, 2026
Merged

[Feature:InstructorUI] Show username on create course page#13025
bmcutler merged 1 commit into
mainfrom
username-on-create-course

Conversation

@roye2

@roye2 roye2 commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Why is this Change Important & Necessary?

It would be useful to instructors / sysadmins to know which user they are logged in as when they are creating a course.

What is the New Behavior?

image

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

  1. on main, verify that the username is not displayed on the create course page
  2. on the branch, verify that the username is properly displayed on the create course page

Automated Testing & Documentation

This is not covered by e2e testing and doesn't need to be.

Other information

this is not a breaking change.

@codecov

codecov Bot commented Jul 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 21.29%. Comparing base (c603aab) to head (42ed629).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff            @@
##               main   #13025   +/-   ##
=========================================
  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.

@mattLif
mattLif self-requested a review July 10, 2026 18:30
@automateprojectmangement automateprojectmangement Bot moved this from Seeking Reviewer to In Review in Submitty Development Jul 10, 2026

@mattLif mattLif 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.

Verified that username appears on the create course page after applying changes.

@github-project-automation github-project-automation Bot moved this from In Review to Awaiting Maintainer Review in Submitty Development Jul 10, 2026
@bmcutler
bmcutler merged commit 161a27f into main Jul 11, 2026
55 of 56 checks passed
@bmcutler
bmcutler deleted the username-on-create-course branch July 11, 2026 18:50
@github-project-automation github-project-automation Bot moved this from Awaiting Maintainer Review to Done in Submitty Development Jul 11, 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.

3 participants