Create `SyncOrchestrator`#4176

Merged

jmartinesp merged 15 commits intodevelopfrom

refactor/sync-observer

Feb 6, 2025

Commits on Feb 5, 2025

Create `SyncOrchestrator` to centralise the sync start/stop flow through the whole app.
jmartinesp
committed
Fix errors after rebase
jmartinesp
committed
Make network monitor return network connectivity status, not internet connectivity
jmartinesp
committed
Don't stop the `SyncService` when network connection is lost, let it fail instead
jmartinesp
committed
Rename `NetworkStatus.Online/Offline` to `Connected/Disconnected` so they're not easily mistaken with internet connectivity instead
jmartinesp
committed
Remove `DefaultSyncOrchestrator.stop()`
jmartinesp
committed
Start the SyncOrchestrator as soon as the Client is created
jmartinesp
committed
Remove the force initial sync, the orchestrator should handle that in the existing loop when offline mode is enabled for the Client in the SDK
jmartinesp
committed
Remove `SyncOrchestrator` interface, rename `DefaultSyncOrchestrator` to it, remove the existing provider interface.
jmartinesp
committed
Use the right coroutine scope for `SyncOrchestrator`: before the app coroutine scope was being incorrectly injected instead.
jmartinesp
committed
Add test to ensure the delay for stopping the sync is honoured
jmartinesp
committed