Skip to content

Dev: Fix useExhaustiveDependencies — Mobile ASA-Go #5529

@conbrad

Description

@conbrad

Describe the task
Remove biome-ignore lint/correctness/useExhaustiveDependencies comments and properly fix dep arrays in the ASA-Go mobile app.

Acceptance Criteria

  • Remove biome-ignore comments from mobile/asa-go/src/App.tsx (lines 109, 127)
  • Remove biome-ignore comments from mobile/asa-go/src/components/map/ASAGoMap.tsx (lines 212, 221, 241, 257, 275)
  • Remove biome-ignore comments from mobile/asa-go/src/components/map/ScaleContainer.tsx (line 10)
  • Remove biome-ignore comments from mobile/asa-go/src/components/settings/Settings.tsx (line 48)
  • Remove biome-ignore comments from mobile/asa-go/src/hooks/usePushNotifications.ts (line 115)
  • Biome lint passes with zero `useExhaustiveDependencies` errors — run from repo root: `./web/node_modules/.bin/biome lint mobile/asa-go/src/`
  • Map initializes, layers render, and scale updates correctly in the mobile app
  • Network status changes correctly toggle basemap visibility
  • Push notification registration works correctly

Additional context

  • Must lint from the repo root (not from `web/`), as `biome.json` `includes` patterns are relative to the repo root.
  • `ASAGoMap.tsx` has 5 affected effects — review carefully for OL layer lifecycle, similar to the shared Map components task.

Metadata

Metadata

Assignees

No one assigned

    Labels

    TaskTechnical task that cannot be written as a user story

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions