Skip to content

SF-2101 Fix errors continuing after project deletion dialog shown - #4046

Merged
pmachapman merged 1 commit into
masterfrom
fix/SF-2101-deletion-dialog-allows-errors
Aug 16, 2026
Merged

SF-2101 Fix errors continuing after project deletion dialog shown#4046
pmachapman merged 1 commit into
masterfrom
fix/SF-2101-deletion-dialog-allows-errors

Conversation

@Nateowami

@Nateowami Nateowami commented Aug 14, 2026

Copy link
Copy Markdown
Collaborator

This change is Reviewable

@codecov

codecov Bot commented Aug 14, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.08%. Comparing base (4f0e8e2) to head (a44c302).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4046   +/-   ##
=======================================
  Coverage   81.08%   81.08%           
=======================================
  Files         661      661           
  Lines       43060    43061    +1     
  Branches     7062     7036   -26     
=======================================
+ Hits        34916    34917    +1     
- Misses       6972     6987   +15     
+ Partials     1172     1157   -15     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

@Nateowami Nateowami added testing complete Testing of PR is complete and should no longer hold up merging of the PR and removed ready to test labels Aug 14, 2026
@Nateowami
Nateowami marked this pull request as ready for review August 14, 2026 14:21
@pmachapman
pmachapman force-pushed the fix/SF-2101-deletion-dialog-allows-errors branch from f12ae9f to a44c302 Compare August 16, 2026 21:08
@pmachapman pmachapman self-assigned this Aug 16, 2026

@pmachapman pmachapman left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

:lgtm:

@pmachapman reviewed 2 files and all commit messages, and made 1 comment.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on Nateowami).

@pmachapman
pmachapman enabled auto-merge (squash) August 16, 2026 21:11
@pmachapman
pmachapman deployed to screenshot_diff August 16, 2026 21:16 — with GitHub Actions Active
@pmachapman
pmachapman merged commit c4e4f30 into master Aug 16, 2026
28 checks passed
@pmachapman
pmachapman deleted the fix/SF-2101-deletion-dialog-allows-errors branch August 16, 2026 21:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

testing complete Testing of PR is complete and should no longer hold up merging of the PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants