Skip to content

Sidebar fixes

Sidebar fixes #1147

Triggered via push December 7, 2023 11:32
Status Success
Total duration 33s
Artifacts
Run Lint Checks
25s
Run Lint Checks
Fit to window
Zoom out
Zoom in

Annotations

5 warnings
Run Lint Checks: src/OnboardingSPA/components/Sidebar/components/SitegenEditorPatterns/Sidebar.js#L112
React Hook useEffect has a missing dependency: 'loadData'. Either include it or remove the dependency array
Run Lint Checks: src/OnboardingSPA/components/Sidebar/components/SitegenEditorPatterns/Sidebar.js#L178
React Hook useEffect has missing dependencies: 'globalStyles', 'handleFavorite', and 'handlePreview'. Either include them or remove the dependency array. You can also replace multiple useState variables with useReducer if 'setActiveTab' needs the current value of 'globalStyles'
Run Lint Checks: src/OnboardingSPA/components/TabPanelHover/index.js#L71
React Hook useEffect has missing dependencies: 'initialTabName' and 'selected'. Either include them or remove the dependency array. If 'setSelected' needs the current value of 'initialTabName', you can also switch to useReducer instead of useState and read 'initialTabName' in the reducer
Run Lint Checks: src/OnboardingSPA/steps/SiteGen/Editor/index.js#L43
React Hook useEffect has a missing dependency: 'loadData'. Either include it or remove the dependency array
Run Lint Checks: src/OnboardingSPA/steps/SiteGen/Preview/index.js#L81
React Hook useEffect has missing dependencies: 'loadData', 'setDrawerActiveView', 'setHeaderActiveView', 'setIsHeaderEnabled', and 'setSidebarActiveView'. Either include them or remove the dependency array