refactor(ui): migrate SearchDropdown consumers to FilterSelect, delete SearchDropdown #29438
Triggered via pull request
September 9, 2026 13:23
harsh-vador
harsh-vador
synchronize
#33034
Status
Cancelled
Total duration
4m 59s
Artifacts
–
Annotations
6 errors and 11 warnings
|
checkstyle
Canceling since a higher priority waiting request for ui-checkstyle-feat-searchdropdown-consumers exists
|
|
checkstyle
The operation was canceled.
|
|
checkstyle:
openmetadata-ui/src/main/resources/ui/src/components/observability/common/FilterChip/FilterChip.test.tsx#L58
Extract this nested ternary operation into an independent statement
|
|
checkstyle:
openmetadata-ui/src/main/resources/ui/src/components/observability/common/FilterChip/FilterChip.test.tsx#L41
Reduce the number of conditional operators (4) used in the expression (maximum allowed 3)
|
|
UI Checkstyle
Canceling since a higher priority waiting request for ui-checkstyle-feat-searchdropdown-consumers exists
|
|
ui-checkstyle
Process completed with exit code 1.
|
|
authorize
Cache reservation failed: cache write denied: token has no writable scopes
|
|
checkstyle:
openmetadata-ui/src/main/resources/ui/src/components/Database/TableQueries/TableQueries.tsx#L488
React Hook useEffect has missing dependencies: 'fetchFilteredQueries', 'isTableDeleted', and 'searchParams.queryFrom'. Either include them or remove the dependency array
|
|
checkstyle:
openmetadata-ui/src/main/resources/ui/src/components/Database/TableQueries/TableQueries.tsx#L204
Review these sequential API requests. If they are independent, start them together with Promise.all/Promise.allSettled; keep sequencing only when data-dependent
|
|
checkstyle:
openmetadata-ui/src/main/resources/ui/src/components/Database/TableQueries/TableQueries.tsx#L179
React Hook useEffect has a missing dependency: 'fetchResourcePermission'. Either include it or remove the dependency array
|
|
checkstyle:
openmetadata-ui/src/main/resources/ui/src/components/DataQuality/DataQualityDashboard/DataQualityDashboard.test.tsx#L23
Pages are route-level composition modules. Move the shared implementation/type to a lower layer instead of importing a page from here
|
|
checkstyle:
openmetadata-ui/src/main/resources/ui/src/components/DataQuality/DataQualityDashboard/DataQualityDashboard.component.tsx#L109
Use <Tooltip> from @openmetadata/ui-core-components instead of raw title="" attributes for consistent tooltip behavior
|
|
checkstyle:
openmetadata-ui/src/main/resources/ui/src/components/DataQuality/DataQualityDashboard/DataQualityDashboard.component.tsx#L33
This runtime import participates in a circular dependency. Extract the shared type/constant/utility or invert the dependency
|
|
checkstyle:
openmetadata-ui/src/main/resources/ui/src/components/DataQuality/DataQualityDashboard/DataQualityDashboard.component.tsx#L30
This runtime import participates in a circular dependency. Extract the shared type/constant/utility or invert the dependency
|
|
checkstyle:
openmetadata-ui/src/main/resources/ui/src/components/DataQuality/DataQualityDashboard/DataQualityDashboard.component.tsx#L23
This runtime import participates in a circular dependency. Extract the shared type/constant/utility or invert the dependency
|
|
checkstyle:
openmetadata-ui/src/main/resources/ui/src/components/DataQuality/DataQualityDashboard/DataQualityDashboard.component.tsx#L22
This runtime import participates in a circular dependency. Extract the shared type/constant/utility or invert the dependency
|
|
checkstyle:
openmetadata-ui/src/main/resources/ui/src/components/AuditLog/AuditLogFilters.component.tsx#L135
React Hook useMemo has an unnecessary dependency: 't'. Either exclude it or remove the dependency array
|