Test

🐛 (mobile) fix amount input requiring two clicks on safari mobile #12749

Sign in to view logs

Triggered via pull request January 17, 2025 20:12

MatissJanis

synchronize #4182

matiss/fix-2392

Status Failure

Total duration 1m 21s

Artifacts –

check.yml

on: pull_request

Annotations

1 error and 5 warnings

Process completed with exit code 1.

ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636

ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636

lint: packages/desktop-client/src/components/App.tsx#L149

React Hook useEffect has a missing dependency: 'dispatch'. Either include it or remove the dependency array. Outer scope values like 'addNotification' aren't valid dependencies because mutating them doesn't re-render the component

ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636

ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636