feat: migrate fills table to abacus-ts #5189
tylerooooAnnotations
11 warnings
|
lint
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
|
|
lint:
src/abacus-ts/rest/orders.ts#L43
Unnecessary conditional, expected left-hand side of `??` operator to be possibly null or undefined
|
|
lint:
src/components/Button.tsx#L228
Unnecessary conditional, value is always truthy
|
|
lint:
src/components/ComboboxMenu.tsx#L203
Unnecessary conditional, expected left-hand side of `??` operator to be possibly null or undefined
|
|
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/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
|
|
lint:
src/components/QrCode.tsx#L73
React Hook useEffect has a missing dependency: 'qrCode'. Either include it or remove the dependency array
|
|
lint:
src/components/QrCode.tsx#L77
React Hook useEffect has a missing dependency: 'size'. Either include it or remove the dependency array. Mutable values like 'ref.current' aren't valid dependencies because mutating them doesn't re-render the component
|