Gino Miceli

[Code health] Stop catching `Throwable`

1

If we need to catch all errors, catch Exception instead. Also, we should avoid using Kotlin `error()` except for unrecoverable exceptions. We may also want to avoid using `runCatching`, as...

This occurs if the first attempt times out. Increasing the timeout should fix this.

There are cases where we may accidentally be replaying navigation events after config change because we've incorrectly modeled them as `LiveData` or `StateFlow`s. We should correct those cases ASAP.

[Data sync] Cancel workers when removing offline surveys or signing out

2

The fact that workers might still be running could lead to db inconsistency. Before doing anything else, we should cancel the running workers for all LOIs in the survey being...

Surveys with no predefined LOIs and no ad hoc jobs are unusable

5

We should show an error message in the app in this case

Or is the Android app only causing problems for organizers when there are >10k polygons? If so, instruct peole collecting data to create a separate account for data collection and...