Skip to content

Merge branch 'alpha' into back/main #324

Merge branch 'alpha' into back/main

Merge branch 'alpha' into back/main #324

Triggered via push November 26, 2024 09:42
Status Success
Total duration 41s
Artifacts

CI.yml

on: push
BE-test-and-build
32s
BE-test-and-build
FE-test-and-build
17s
FE-test-and-build
Fit to window
Zoom out
Zoom in

Annotations

3 warnings
FE-test-and-build: FE/src/components/Search/index.tsx#L37
React Hook useEffect has missing dependencies: 'addSearchHistory' and 'isLoading'. Either include them or remove the dependency array
FE-test-and-build: FE/src/components/StocksDetail/Chart.tsx#L263
React Hook useCallback has unnecessary dependencies: 'drawBarChart', 'drawCandleChart', 'drawChartGrid', 'drawLineChart', 'drawLowerYAxis', 'drawUpperYAxis', 'drawXAxis', and 'padding'. Either exclude them or remove the dependency array. Outer scope values like 'padding' aren't valid dependencies because mutating them doesn't re-render the component
FE-test-and-build: FE/src/components/StocksDetail/PriceSection.tsx#L34
React Hook useCallback has a missing dependency: 'queryClient'. Either include it or remove the dependency array