-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
feat(ingestion) cleaning up ingestion page and aligning with SaaS #12710
base: master
Are you sure you want to change the base?
Conversation
@@ -61,6 +54,29 @@ function IngestionSourceTable({ | |||
}: Props) { | |||
const isShowNavBarRedesign = useShowNavBarRedesign(); | |||
|
|||
const tableData = sources.map((source) => ({ |
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.
moved this up above columns so we can reference its type
@@ -83,20 +99,7 @@ function IngestionSourceTable({ | |||
render: ScheduleColumn, | |||
}, | |||
{ | |||
title: 'Execution Count', |
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.
consolidated these columns under status
❌ 2 Tests Failed:
View the top 1 failed test(s) by shortest run time
View the full list of 1 ❄️ flaky tests
To view more test analytics, go to the Test Analytics Dashboard |
Checklist