Merge pull request #463 from newfold-labs/enhance/some-more-fixes-now #1688
Annotations
10 warnings
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, actions/setup-node@v3, actions/cache@v3, technote-space/get-diff-action@v6. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
Run JS Lint:
src/OnboardingSPA/components/Loaders/SiteGenLoader/index.js#L24
Assignments to the 'statusIdx' variable from inside React Hook useEffect will be lost after each render. To preserve the value over time, store it in a useRef Hook and keep the mutable value in the '.current' property. Otherwise, you can move this variable directly inside useEffect
|
Run JS Lint:
src/OnboardingSPA/components/Loaders/SiteGenLoader/index.js#L41
React Hook useEffect has a missing dependency: 'currentData?.sitegen?.siteGenMetaStatus?.totalCount'. Either include it or remove the dependency array
|
Run JS Lint:
src/OnboardingSPA/components/Loaders/SiteGenLoader/index.js#L52
React Hook useEffect has missing dependencies: 'customNavPercentage', 'navigate', and 'nextStep'. Either include them or remove the dependency array
|
Run JS Lint:
src/OnboardingSPA/steps/SiteGen/Experience/index.js#L33
React Hook useEffect contains a call to 'setSelection'. Without a list of dependencies, this can lead to an infinite chain of updates. To fix this, pass [setHideFooterNav, setIsHeaderEnabled, setSidebarActiveView, setHeaderActiveView, setDrawerActiveView, currentData.sitegen.experience.level] as a second argument to the useEffect Hook
|
Run JS Lint:
src/OnboardingSPA/steps/SiteGen/Preview/index.js#L58
React Hook useEffect has missing dependencies: 'setDrawerActiveView', 'setHeaderActiveView', 'setHideFooterNav', 'setIsHeaderEnabled', 'setSidebarActiveView', and 'updateInitialize'. Either include them or remove the dependency array
|
Run JS Lint:
src/OnboardingSPA/steps/SiteGen/Preview/index.js#L69
React Hook useEffect has a missing dependency: 'loadHomepages'. Either include it or remove the dependency array
|
Run JS Lint:
src/OnboardingSPA/steps/SiteGen/Preview/index.js#L111
React Hook useEffect has a missing dependency: 'loadHomepages'. Either include it or remove the dependency array
|
Run JS Lint:
src/OnboardingSPA/steps/SiteGen/SiteLogo/index.js#L68
React Hook useEffect has missing dependencies: 'currentData.data.siteLogo', 'getEditedEntityRecord', 'setDrawerActiveView', 'setHeaderActiveView', 'setHideFooterNav', 'setIsFooterNavAllowed', 'setIsHeaderEnabled', and 'setSidebarActiveView'. Either include them or remove the dependency array
|
Run JS Lint:
src/OnboardingSPA/steps/SiteGen/SocialMedia/index.js#L57
React Hook useEffect has missing dependencies: 'navigate', 'nextStep.path', and 'setIsFooterNavAllowed'. Either include them or remove the dependency array
|
The logs for this run have expired and are no longer available.
Loading