UI Checkstyle #29424
karanh37Annotations
2 warnings
|
checkstyle:
openmetadata-ui/src/main/resources/ui/playwright/e2e/Pages/Domains.spec.ts#L113
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/DataProduct/DataProductListPage.tsx#L392
React Hook useMemo has missing dependencies: 'dataProductColumns', 'dataProductListing.handleSelect', and 'dataProductListing.handleSelectAll'. Either include them or remove the dependency array
|