Skip to content
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 tabs to evaluation dialog #26960

Merged
merged 1 commit into from
Jan 14, 2025
Merged

Conversation

hellendag
Copy link
Member

@hellendag hellendag commented Jan 8, 2025

Summary & Motivation

Update EvaluationDetailDialog to have two tabs: Evaluation and Runs. The first shows what the dialog currently shows, which is the details of the evaluation itself. The second is a filtered Runs table with the runs associated with the evaluation.

Screenshot 2025-01-14 at 10 52 53 Screenshot 2025-01-14 at 10 52 48

How I Tested These Changes

View dialog from automation sensor and asset automations list. Verify that the tabs behave and render correctly.

Changelog

[ui] When viewing details of an automation evaluation, show the relevant runs in a tab within the same dialog.

Copy link
Member Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

Copy link

github-actions bot commented Jan 8, 2025

Deploy preview for dagit-core-storybook ready!

✅ Preview
https://dagit-core-storybook-1uudf8l8w-elementl.vercel.app
https://dish-tabbed-evaluation-dialog.core-storybook.dagster-docs.io

Built with commit 2efd92d.
This pull request is being automatically deployed with vercel-action

@hellendag hellendag requested a review from OwenKephart January 9, 2025 16:40
@hellendag hellendag force-pushed the dish/tabbed-evaluation-dialog branch 2 times, most recently from 9e22794 to e1a95d6 Compare January 10, 2025 15:29
@hellendag hellendag marked this pull request as ready for review January 14, 2025 16:41
@hellendag hellendag force-pushed the dish/tabbed-evaluation-dialog branch from e1a95d6 to 2efd92d Compare January 14, 2025 16:46
@hellendag hellendag force-pushed the dish/tabbed-evaluation-dialog branch from 2efd92d to fead83e Compare January 14, 2025 17:07
Copy link
Member Author

hellendag commented Jan 14, 2025

Merge activity

  • Jan 14, 3:08 PM CST: A user started a stack merge that includes this pull request via Graphite.
  • Jan 14, 3:09 PM CST: A user merged this pull request with Graphite.

@hellendag hellendag merged commit b9334d6 into master Jan 14, 2025
1 of 2 checks passed
@hellendag hellendag deleted the dish/tabbed-evaluation-dialog branch January 14, 2025 21:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants