-
Notifications
You must be signed in to change notification settings - Fork 359
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
feat: show metadata in run table #9776
Conversation
✅ Deploy Preview for determined-ui ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
91aa01b
to
89ad92e
Compare
fixed it |
@@ -18,3 +18,9 @@ | |||
white-space: nowrap; | |||
} | |||
} | |||
|
|||
@media only screen and (width <= 390px) { |
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.
looks like $breakpoint-
variables are not working so use the constant number
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #9776 +/- ##
==========================================
- Coverage 54.04% 49.73% -4.32%
==========================================
Files 1260 937 -323
Lines 155551 126228 -29323
Branches 3501 3502 +1
==========================================
- Hits 84073 62779 -21294
+ Misses 71332 63303 -8029
Partials 146 146
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
Ticket
ET-222
ET-223
Description
Show metadata in the run table.
have not excluded the array type columns yet. will implement it after #9759 is merged, but its not a blocker to merge this PR.
Test Plan
Columns
button and seeMetadata
tab. user should be able to add metadata columns and filter and sort by metadata (metadata with array type are not supported for filtering and sorting, PR will come after feat: add array column type for abitrary metadata #9759 to exclude the array type metadata in filter and sort)Checklist
docs/release-notes/
See Release Note for details.