Skip to content

[Bugfix:InstructorUI] Phantom files notebook sub#13009

Merged
bmcutler merged 11 commits into
mainfrom
fix-triple-submission
Jul 8, 2026
Merged

[Bugfix:InstructorUI] Phantom files notebook sub#13009
bmcutler merged 11 commits into
mainfrom
fix-triple-submission

Conversation

@dagemcn

@dagemcn dagemcn commented Jul 7, 2026

Copy link
Copy Markdown
Member

Why is this Change Important & Necessary?

Closes Issue#13001

What is the New Behavior?

Prior to the changes, the notebook builder submission box would show 3 files when you had previously submitted just one.
image
After the changes, the submission box only shows the one file that was submitted.
Screenshot 2026-07-07 at 11 24 52 AM

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

  1. Go to any gradeable with a notebook file submission component (notebook_filesubmission in sample)
  2. Submit a file in the submission box
  3. Observe error

Automated Testing & Documentation

This feature is not tested by E2E tests.

@github-project-automation github-project-automation Bot moved this to Seeking Reviewer in Submitty Development Jul 7, 2026
@automateprojectmangement automateprojectmangement Bot moved this from Seeking Reviewer to Work in Progress in Submitty Development Jul 7, 2026
@dagemcn dagemcn changed the title [Feature:InstructorUI] Triple submission UI bug [Feature:InstructorUI] Fix notebook builder submission UI bug Jul 7, 2026
@dagemcn dagemcn changed the title [Feature:InstructorUI] Fix notebook builder submission UI bug [Bugfix:InstructorUI] Fix notebook builder submission UI bug Jul 7, 2026
@codecov

codecov Bot commented Jul 7, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 21.29%. Comparing base (6582e94) to head (152221b).

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##               main   #13009      +/-   ##
============================================
- Coverage     21.29%   21.29%   -0.01%     
  Complexity    10071    10071              
============================================
  Files           278      278              
  Lines         37697    37698       +1     
  Branches        516      516              
============================================
  Hits           8028     8028              
- Misses        29166    29167       +1     
  Partials        503      503              
Flag Coverage Δ
autograder 21.16% <ø> (ø)
js 1.98% <ø> (ø)
migrator 100.00% <ø> (ø)
php 20.20% <0.00%> (-0.01%) ⬇️
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 changed the title [Bugfix:InstructorUI] Fix notebook builder submission UI bug [Bugfix:InstructorUI] Fix notebook submission UI bug Jul 7, 2026
@dagemcn dagemcn changed the title [Bugfix:InstructorUI] Fix notebook submission UI bug [Bugfix:InstructorUI] Notebook submission UI bug Jul 7, 2026
@dagemcn dagemcn changed the title [Bugfix:InstructorUI] Notebook submission UI bug [Bugfix:InstructorUI] Notebook file submission bug Jul 7, 2026
@dagemcn dagemcn changed the title [Bugfix:InstructorUI] Notebook file submission bug [Bugfix:InstructorUI] Notebook file submission bug Jul 7, 2026
@dagemcn dagemcn changed the title [Bugfix:InstructorUI] Notebook file submission bug [Bugfix:InstructorUI] Notebook file submission bug Jul 7, 2026
@dagemcn dagemcn changed the title [Bugfix:InstructorUI] Notebook file submission bug [Bugfix:Submission] Notebook file submission UI bug Jul 7, 2026
@dagemcn dagemcn changed the title [Bugfix:Submission] Notebook file submission UI bug [Bugfix:InstructorUI] Notebook file submission Jul 7, 2026
@dagemcn
dagemcn marked this pull request as ready for review July 7, 2026 15:42
@automateprojectmangement automateprojectmangement Bot moved this from Work in Progress to Seeking Reviewer in Submitty Development Jul 7, 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.

Tested again with same process as PR#13002, and the triple file appearance bug is gone now.

@github-project-automation github-project-automation Bot moved this from Seeking Reviewer to Awaiting Maintainer Review in Submitty Development Jul 7, 2026
@dagemcn dagemcn changed the title [Bugfix:InstructorUI] Notebook file submission [Bugfix:InstructorUI] Phantom files notebook sub Jul 7, 2026
@bmcutler
bmcutler merged commit fa569a0 into main Jul 8, 2026
52 of 53 checks passed
@bmcutler
bmcutler deleted the fix-triple-submission branch July 8, 2026 19:47
parthrajsinghbhati pushed a commit to parthrajsinghbhati/Submitty that referenced this pull request Jul 10, 2026
### Why is this Change Important & Necessary?
Closes [Issue#13001](Submitty#13001)

### What is the New Behavior?
Prior to the changes, the notebook builder submission box would show 3
files when you had previously submitted just one.
<img width="2280" height="846" alt="image"
src="https://github.com/user-attachments/assets/159e9d4b-16ae-4764-814e-e33ed3940516"
/>
After the changes, the submission box only shows the one file that was
submitted.
<img width="1372" height="456" alt="Screenshot 2026-07-07 at 11 24
52 AM"
src="https://github.com/user-attachments/assets/ef324f0a-9eaa-40ad-bfd3-c15475e00bb4"
/>

### What steps should a reviewer take to reproduce or test the bug or
new feature?
1. Go to any gradeable with a notebook file submission component
(notebook_filesubmission in sample)
2. Submit a file in the submission box
3. Observe error

### Automated Testing & Documentation
This feature is not tested by E2E tests.

---------

Co-authored-by: Barb Cutler <[email protected]>
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.

notebook file submission box visual bug

3 participants