-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
[ui] add catalog pill for row count #21867
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. Join @benpankow and the rest of your teammates on Graphite |
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!
but will defer to others on the actual frontend code
const rowCountMeta: IntMetadataEntry | undefined = materialization?.metadataEntries.find( | ||
(entry) => isCanonicalRowCountMetadataEntry(entry), | ||
) as IntMetadataEntry | undefined; | ||
|
||
const renderStatusSection = () => ( | ||
<Box flex={{direction: 'column', gap: 16}}> | ||
<Box flex={{direction: 'row'}}> |
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.
display: grid;
grid-template-columns: repeat(3, minmax(0, 1fr));
d856900
to
52887e7
Compare
## Summary Adds an asset catalog pill for the special `dagster/row_count` metadata entry based on designs. <img width="789" alt="Screenshot 2024-05-15 at 10 01 22 AM" src="https://github.com/dagster-io/dagster/assets/10215173/bd3f839b-3b2b-4d87-8110-e0e3f068ab32"> ## Test Plan Tested locally.
Summary
Adds an asset catalog pill for the special
dagster/row_count
metadata entry based on designs.Test Plan
Tested locally.