-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Asset tags in the UI #20389
Asset tags in the UI #20389
Conversation
Deploy preview for dagit-core-storybook ready! ✅ Preview Built with commit 03e3cab. |
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.
JS looks fine to me. Can you also add a screenshot?
2dfe80e
to
a6b561c
Compare
4dd3b34
to
ccf1bbc
Compare
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.
Will defer to the UI folks on this one
ccf1bbc
to
b0b0550
Compare
@hellendag - added a screenshot to the PR description. I haven't handled individual tags that exceed the width, and was hoping to pass that off to someone on the frontend team as a followup. |
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.
That sounds fine. Thanks for the screenshot!
b0b0550
to
29cb275
Compare
1e281c8
to
67c0e8c
Compare
branch-name: graphql-asset-tags
67c0e8c
to
03e3cab
Compare
## Summary & Motivation This PR passes tags up through graphql to the UI and displays them on the asset overview page. It doesn't yet support the no-value tags introduced in this PR: #20388. Stacks on top of #20351. <img width="1167" alt="image" src="https://github.com/dagster-io/dagster/assets/654855/97bcb447-06e6-4cd1-a0f2-4fdfffdd1d0d"> ## How I Tested These Changes
Summary & Motivation
This PR passes tags up through graphql to the UI and displays them on the asset overview page. It doesn't yet support the no-value tags introduced in this PR: #20388.
Stacks on top of #20351.
How I Tested These Changes