farrah/WEBREL-5099/fix: full page modals #1564
Triggered via pull request
October 7, 2025 11:28
farrah-deriv
synchronize
#651
Status
Success
Total duration
2m 0s
Artifacts
–
build-and-deploy-test.yml
on: pull_request_target
build_to_cloudflare_pages
1m 56s
Annotations
10 warnings
build_to_cloudflare_pages:
src/hooks/custom-hooks/useModalManager.ts#L68
React Hook useEffect has a missing dependency: 'syncModalParams'. Either include it or remove the dependency array
|
build_to_cloudflare_pages:
src/hooks/custom-hooks/useGrowthbookGetFeatureValue.tsx#L31
React Hook useEffect has missing dependencies: 'featureFlag' and 'resolvedDefaultValue'. Either include them or remove the dependency array
|
build_to_cloudflare_pages:
src/hooks/custom-hooks/useGetHubEnabledCountryList.ts#L28
React Hook useEffect has missing dependencies: 'HUB_REMOTE_CONFIG_URL', 'getRemoteConfig', and 'hubEnabledCountryList.length'. Either include them or remove the dependency array
|
build_to_cloudflare_pages:
src/hooks/custom-hooks/useGetHubEnabledCountryList.ts#L24
React Hook useCallback has missing dependencies: 'HUB_REMOTE_CONFIG_URL' and 'setHubEnabledCountryList'. Either include them or remove the dependency array
|
build_to_cloudflare_pages:
src/components/Table/Table.tsx#L45
React Hook useLayoutEffect has an unnecessary dependency: 'headerRef.current'. Either exclude it or remove the dependency array. Mutable values like 'headerRef.current' aren't valid dependencies because mutating them doesn't re-render the component
|
build_to_cloudflare_pages:
src/components/Modals/FilterModal/FilterModalPaymentMethods/FilterModalPaymentMethods.tsx#L38
React Hook useEffect has a missing dependency: 'searchedPaymentMethods'. Either include it or remove the dependency array
|
build_to_cloudflare_pages:
src/components/AppHeader/Notifications/Notifications.tsx#L39
React Hook useMemo has missing dependencies: 'history', 'isDesktop', and 'localize'. Either include them or remove the dependency array
|
build_to_cloudflare_pages:
src/components/AppHeader/MobileMenu/MobileMenuConfig.tsx#L50
Do not nest ternary expressions
|
build_to_cloudflare_pages:
src/components/AppHeader/CurrencyIcon/CurrencyIcon.tsx#L26
React Hook useEffect has missing dependencies: 'currency' and 'isVirtual'. Either include them or remove the dependency array
|
build_to_cloudflare_pages:
src/App.tsx#L53
React Hook useEffect has missing dependencies: 'isTMBEnabled' and 'isTMBInitialized'. Either include them or remove the dependency array
|