-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
remove compute log manager UI (#23422)
## Summary & Motivation We should always have a compute log manager implementing the `CapturedLogManager` interface, so we can get rid of all the UI code in the non-captured log manager branch. ## How I Tested These Changes BK, loaded run page
- Loading branch information
Showing
20 changed files
with
32 additions
and
704 deletions.
There are no files selected for viewing
24 changes: 0 additions & 24 deletions
24
js_modules/dagster-ui/packages/ui-core/src/graphql/schema.graphql
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
105 changes: 0 additions & 105 deletions
105
js_modules/dagster-ui/packages/ui-core/src/graphql/types.ts
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
10 changes: 0 additions & 10 deletions
10
js_modules/dagster-ui/packages/ui-core/src/instance/types/useSupportsCapturedLogs.types.ts
This file was deleted.
Oops, something went wrong.
25 changes: 0 additions & 25 deletions
25
js_modules/dagster-ui/packages/ui-core/src/instance/useSupportsCapturedLogs.tsx
This file was deleted.
Oops, something went wrong.
Oops, something went wrong.
b3a0be3
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.
Deploy preview for dagit-core-storybook ready!
✅ Preview
https://dagit-core-storybook-5nzhu6uqa-elementl.vercel.app
Built with commit b3a0be3.
This pull request is being automatically deployed with vercel-action