Skip to content

task/WG-401-Add-Asset-to-Feature-Hook #849

task/WG-401-Add-Asset-to-Feature-Hook

task/WG-401-Add-Asset-to-Feature-Hook #849

Triggered via pull request January 29, 2025 22:46
Status Success
Total duration 2m 2s
Artifacts

main.yml

on: pull_request
Angular-Unit-Tests
57s
Angular-Unit-Tests
Angular-Linting
56s
Angular-Linting
React-Unit-Tests
1m 51s
React-Unit-Tests
React-Linting
33s
React-Linting
React-Build
57s
React-Build
Fit to window
Zoom out
Zoom in

Annotations

8 warnings
React-Linting: react/src/components/Files/FileListing.tsx#L74
React Hook useEffect has a missing dependency: 'selectedSystem'. Either include it or remove the dependency array
React-Linting: react/src/components/Files/FileListing.tsx#L92
React Hook useEffect has a missing dependency: 'allowedFileExtensions'. Either include it or remove the dependency array. If 'setChonkyFiles' needs the current value of 'allowedFileExtensions', you can also switch to useReducer instead of useState and read 'allowedFileExtensions' in the reducer
React-Linting: react/src/components/Files/FileListing.tsx#L163
React Hook useCallback has missing dependencies: 'onFileSelect', 'onFolderSelect', and 'selectedFiles'. Either include them or remove the dependency array. If 'onFolderSelect' changes too often, find the parent component that defines it and wrap that definition in useCallback
React-Linting: react/src/components/Systems/SystemSelect.tsx#L33
React Hook useEffect has a missing dependency: 'onSystemSelect'. Either include it or remove the dependency array. If 'onSystemSelect' changes too often, find the parent component that defines it and wrap that definition in useCallback
React-Linting: react/src/hooks/systems/useSystems.ts#L18
The 'systems' logical expression could make the dependencies of useMemo Hook (at line 22) change on every render. To fix this, wrap the initialization of 'systems' in its own useMemo() Hook
React-Linting: react/src/hooks/systems/useSystems.ts#L18
The 'systems' logical expression could make the dependencies of useMemo Hook (at line 28) change on every render. To fix this, wrap the initialization of 'systems' in its own useMemo() Hook
React-Linting: react/src/hooks/systems/useSystems.ts#L18
The 'systems' logical expression could make the dependencies of useMemo Hook (at line 34) change on every render. To fix this, wrap the initialization of 'systems' in its own useMemo() Hook
React-Linting: react/src/pages/Login/Login.tsx#L29
React Hook useEffect has missing dependencies: 'configuration.geoapiUrl', 'isAuthenticated', 'location.search', and 'navigate'. Either include them or remove the dependency array