fix(leftnavtree): menu is not activated when the URL has pathprefix #559
Triggered via pull request
September 11, 2024 13:19
Status
Success
Total duration
3m 55s
Artifacts
–
validate-and-build.yaml
on: pull_request
validate-and-build
3m 47s
Annotations
9 warnings
validate-and-build:
packages/example/gatsby-ssr.js#L24
Dangerous property 'dangerouslySetInnerHTML' found
|
validate-and-build:
packages/gatsby-theme-carbon/src/components/FeedbackDialog/Experience.js#L18
Component definition is missing display name
|
validate-and-build:
packages/gatsby-theme-carbon/src/components/LeftNav/LeftNav.js#L55
React Hook useEffect has a missing dependency: 'hasNestedLevels'. Either include it or remove the dependency array
|
validate-and-build:
packages/gatsby-theme-carbon/src/components/LeftNav/LeftNavTree.js#L31
'i' is defined but never used. Allowed unused args must match /res|next|^err|^_/u
|
validate-and-build:
packages/gatsby-theme-carbon/src/components/LeftNav/LeftNavTree.js#L76
React Hook useEffect has a missing dependency: 'pathPrefix'. Either include it or remove the dependency array
|
validate-and-build:
packages/gatsby-theme-carbon/src/components/LeftNav/LeftNavTree.js#L106
React Hook useEffect has a missing dependency: 'isTabActive'. Either include it or remove the dependency array
|
validate-and-build:
packages/gatsby-theme-carbon/src/components/LeftNav/LeftNavTree.js#L177
'prevProps' is defined but never used. Allowed unused args must match /res|next|^err|^_/u
|
validate-and-build:
packages/gatsby-theme-carbon/src/components/Tabs/Tabs.js#L141
React Hook useMemo has an unnecessary dependency: 'tabList.current'. Either exclude it or remove the dependency array. Mutable values like 'tabList.current' aren't valid dependencies because mutating them doesn't re-render the component
|
validate-and-build:
packages/gatsby-theme-carbon/src/util/context/NavContext.js#L33
The 'toggleNavState' function makes the dependencies of useMemo Hook (at line 51) change on every render. Move it inside the useMemo callback. Alternatively, wrap the definition of 'toggleNavState' in its own useCallback() Hook
|