Skip to content

feat(fe): confirm modal when leaving settings page #11358

feat(fe): confirm modal when leaving settings page

feat(fe): confirm modal when leaving settings page #11358

Triggered via pull request December 23, 2024 01:46
Status Success
Total duration 3m 52s
Artifacts

ci.yml

on: pull_request
Matrix: Build Backend
Fit to window
Zoom out
Zoom in

Annotations

8 warnings
Build Backend (client)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Build Backend (admin)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Typecheck
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Test Backend
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Lint
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Lint: apps/frontend/app/(client)/(main)/settings/_components/ConfirmNavigation.tsx#L56
React Hook useEffect has missing dependencies: 'bypassConfirmation' and 'updateNow'. Either include them or remove the dependency array. Mutable values like 'bypassConfirmation.current' aren't valid dependencies because mutating them doesn't re-render the component
Lint: apps/frontend/app/(client)/(main)/settings/page.tsx#L156
React Hook useEffect has a missing dependency: 'setValue'. Either include it or remove the dependency array
Build Frontend
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636