Skip to content

Conversation

@jamiedemaria
Copy link
Contributor

@jamiedemaria jamiedemaria commented Nov 11, 2025

Adds the summarize button in oss with the no-op implementation

Copy link
Contributor Author

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

@github-actions
Copy link

github-actions bot commented Nov 11, 2025

Deploy preview for dagit-core-storybook ready!

✅ Preview
https://dagit-core-storybook-a8a2xl443-elementl.vercel.app
https://jamie-ai-summary-runs-list.core-storybook.dagster-docs.io

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

@jamiedemaria jamiedemaria changed the title summary button oss [ui] summarize run menu item Nov 11, 2025
@jamiedemaria jamiedemaria marked this pull request as ready for review November 12, 2025 14:49
@jamiedemaria jamiedemaria force-pushed the jamie/ai-summary-runs-list branch from 76c86d0 to 454709a Compare November 12, 2025 14:57
Comment on lines 3 to 4
// eslint-disable-next-line unused-imports/no-unused-vars
export const AISummaryForRunMenuItem = ({run}: {run: RunActionsMenuRunFragment}) => null;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You should be able to do this to avoid the lint override, since lint should ignore the _-named arg:

export interface Props {
  run: {id: string; status: RunStatus};
}

export const AISummaryForRunMenuItem = (_: Props) => null;

@jamiedemaria jamiedemaria force-pushed the jamie/ai-summary-runs-list branch from 4ef81a2 to ef72030 Compare November 17, 2025 16:37
@jamiedemaria jamiedemaria merged commit 7da0b0a into master Nov 18, 2025
6 checks passed
@jamiedemaria jamiedemaria deleted the jamie/ai-summary-runs-list branch November 18, 2025 16:06
Jongwan93 pushed a commit to Jongwan93/dagster that referenced this pull request Nov 20, 2025
Adds the summarize button in oss with the no-op implementation
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.

3 participants