-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Safeguard discard #459
Safeguard discard #459
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This also needs to apply to the menu-based discard (ClassMethod Revert in SourceControl.Git.Utils) which uses the checkout command rather than restore.
What do you mean by menu-based discard? Where would you access that in the UI? |
Replaced "$$$THROWONERROR" with "$$$ThrowOnError"
Fix file restore for files whose directories have been deleted. Refresh uncommited queue after file discard
Fixed the UI so that the "Restore" button in the discarded files is always visible Run a refresh of the uncommitted queue after the discard
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this looks fine from a code standpoint but want to kick the tires on it before merging. @isc-pbarton could you review too?
Replaced HTML characters with escaped versions, made sure discarded files are compiled after import
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #459 +/- ##
==========================================
- Coverage 25.90% 25.06% -0.84%
==========================================
Files 17 18 +1
Lines 2459 2541 +82
==========================================
Hits 637 637
- Misses 1822 1904 +82 ☔ View full report in Codecov by Sentry. |
fixes #455