feat: cctp withdrawals: clone unchanged components [2/n] #4357
yogurtandjamAnnotations
11 warnings
|
lint
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, dydxprotocol/[email protected], actions/setup-node@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
lint:
src/App.tsx#L116
React Hook useEffect has a missing dependency: 'skipClientId'. Either include it or remove the dependency array
|
|
lint:
src/components/BackButton.tsx#L21
Unnecessary optional chain on a non-nullish value
|
|
lint:
src/components/Button.tsx#L168
Unnecessary conditional, value is always truthy
|
|
lint:
src/components/Details.tsx#L224
Unnecessary conditional, value is always truthy
|
|
lint:
src/components/Details.tsx#L232
Unnecessary conditional, value is always truthy
|
|
lint:
src/components/Details.tsx#L244
Unnecessary conditional, value is always truthy
|
|
lint:
src/components/DropdownMenu.tsx#L82
Unnecessary optional chain on a non-nullish value
|
|
lint:
src/components/FormInput.tsx#L35
Unnecessary optional chain on a non-nullish value
|
|
lint:
src/components/GuardedMobileRoute.tsx#L27
React Hook useEffect has missing dependencies: 'canAccountTrade', 'dispatch', and 'isTablet'. Either include them or remove the dependency array
|
|
lint:
src/components/GuardedMobileRoute.tsx#L38
React Hook useEffect has a missing dependency: 'navigate'. Either include it or remove the dependency array
|