Skip to content

fix: zoom domain excluding first point on data update #4042

fix: zoom domain excluding first point on data update

fix: zoom domain excluding first point on data update #4042

Triggered via pull request October 4, 2024 00:05
Status Success
Total duration 2m 45s
Artifacts

commitlint.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

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/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
lint: src/components/QrCode.tsx#L85
React Hook useEffect has a missing dependency: 'qrCode'. Either include it or remove the dependency array
lint: src/components/TimeoutButton.tsx#L35
React Hook useEffect has a missing dependency: 'onTimeOut'. Either include it or remove the dependency array. If 'onTimeOut' changes too often, find the parent component that defines it and wrap that definition in useCallback
lint: src/components/Toast.tsx#L63
React Hook useEffect has a missing dependency: 'setIsOpen'. Either include it or remove the dependency array. If 'setIsOpen' changes too often, find the parent component that defines it and wrap that definition in useCallback
lint: src/components/visx/TimeSeriesChart.tsx#L235
React Hook useMemo has missing dependencies: 'domainBasePadding', 'earliestDatum', 'latestDatum', 'xAccessor', and 'yAccessor'. Either include them or remove the dependency array
lint: src/components/visx/TimeSeriesChart.tsx#L243
React Hook useEffect has a missing dependency: 'onVisibleDataChange'. Either include it or remove the dependency array. If 'onVisibleDataChange' changes too often, find the parent component that defines it and wrap that definition in useCallback
lint: src/components/visx/XYChartTooltipWithBounds.tsx#L133
React Hook useEffect has a missing dependency: 'onTooltipContext'. Either include it or remove the dependency array. If 'onTooltipContext' changes too often, find the parent component that defines it and wrap that definition in useCallback