refactor(ui): migrate the remaining fixed-feature tables to TableV2 #29477
Triggered via pull request
September 9, 2026 14:34
harsh-vador
harsh-vador
synchronize
#32467
Status
Success
Total duration
5m 45s
Artifacts
–
Annotations
16 warnings
|
authorize
Cache reservation failed: cache write denied: token has no writable scopes
|
|
checkstyle:
openmetadata-ui/src/main/resources/ui/playwright/e2e/Pages/LiveIndexingTab.spec.ts#L75
Prefer the `page` fixture (test.use({ storageState })) over browser.newPage() + manual login for single-user admin tests. For multi-user tests that need a second non-admin page, this warning is expected — no action needed
|
|
checkstyle:
openmetadata-ui/src/main/resources/ui/playwright/e2e/Pages/LiveIndexingTab.spec.ts#L57
Prefer the `page` fixture (test.use({ storageState })) over browser.newPage() + manual login for single-user admin tests. For multi-user tests that need a second non-admin page, this warning is expected — no action needed
|
|
checkstyle:
openmetadata-ui/src/main/resources/ui/playwright/e2e/Pages/DataContracts.spec.ts#L2285
Prefer the `page` fixture (test.use({ storageState })) over browser.newPage() + manual login for single-user admin tests. For multi-user tests that need a second non-admin page, this warning is expected — no action needed
|
|
checkstyle:
openmetadata-ui/src/main/resources/ui/playwright/e2e/Features/OnlineUsers.spec.ts#L237
Prefer the `page` fixture (test.use({ storageState })) over browser.newPage() + manual login for single-user admin tests. For multi-user tests that need a second non-admin page, this warning is expected — no action needed
|
|
checkstyle:
openmetadata-ui/src/main/resources/ui/playwright/e2e/Features/OnlineUsers.spec.ts#L105
Prefer the `page` fixture (test.use({ storageState })) over browser.newPage() + manual login for single-user admin tests. For multi-user tests that need a second non-admin page, this warning is expected — no action needed
|
|
checkstyle:
openmetadata-ui/src/main/resources/ui/src/components/Dashboard/DataModel/DataModels/DataModelsTable.tsx#L253
React Hook useEffect has a missing dependency: 'fetchDashboardsDataModel'. Either include it or remove the dependency array
|
|
checkstyle:
openmetadata-ui/src/main/resources/ui/src/components/Dashboard/DataModel/DataModels/DataModelsTable.tsx#L240
React Hook useEffect has a missing dependency: 'searchDataModels'. Either include it or remove the dependency array
|
|
checkstyle:
openmetadata-ui/src/main/resources/ui/src/components/Dashboard/DataModel/DataModels/DataModelsTable.tsx#L196
React Hook useCallback has a missing dependency: 'handlePagingChange'. Either include it or remove the dependency array
|
|
checkstyle:
openmetadata-ui/src/main/resources/ui/src/components/Dashboard/DataModel/DataModels/DataModelsTable.tsx#L129
React Hook useCallback has a missing dependency: 'pageSize'. Either include it or remove the dependency array
|
|
checkstyle:
openmetadata-ui/src/main/resources/ui/src/components/Dashboard/DataModel/DataModels/DataModelsTable.tsx#L42
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/Classifications/ClassificationDetails/ClassificationDetails.tsx#L553
React Hook useEffect has missing dependencies: 'fetchClassificationChildren', 'handlePageChange', and 'isAddingTag'. Either include them or remove the dependency array
|
|
checkstyle:
openmetadata-ui/src/main/resources/ui/src/components/Classifications/ClassificationDetails/ClassificationDetails.tsx#L503
React Hook useMemo has a missing dependency: 't'. Either include it or remove the dependency array
|
|
checkstyle:
openmetadata-ui/src/main/resources/ui/src/components/Classifications/ClassificationDetails/ClassificationDetails.tsx#L431
React Hook useMemo has a missing dependency: 't'. Either include it or remove the dependency array
|
|
checkstyle:
openmetadata-ui/src/main/resources/ui/src/components/Classifications/ClassificationDetails/ClassificationDetails.tsx#L281
React Hook useCallback has missing dependencies: 'isVersionView' and 'navigate'. Either include them or remove the dependency array
|
|
checkstyle:
openmetadata-ui/src/main/resources/ui/src/components/Classifications/ClassificationDetails/ClassificationDetails.test.tsx#L37
Use <Tooltip> from @openmetadata/ui-core-components instead of raw title="" attributes for consistent tooltip behavior
|