-
Notifications
You must be signed in to change notification settings - Fork 356
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
fix: prevent checkpoint modals from closing on their own [ET-116] [ET-120] #9094
Conversation
✅ Deploy Preview for determined-ui ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #9094 +/- ##
==========================================
- Coverage 47.06% 40.12% -6.95%
==========================================
Files 1154 831 -323
Lines 142378 102998 -39380
Branches 2423 2387 -36
==========================================
- Hits 67006 41324 -25682
+ Misses 75182 61484 -13698
Partials 190 190
Flags with carried forward coverage won't be shown. Click here to find out more.
|
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.
LGTM
Description
Addresses the bugs from release part here https://github.com/determined-ai/release-party-issues/issues/1594
Test Plan
Commentary (optional)
Checklist
docs/release-notes/
.See Release Note for details.
Ticket