Do not cancel current sync request when going to background #5621

Incremental sync can be long and it requires the user to wait for the treatment to end, else all is restarted from the beginning each time the user moves the app to foreground.

changelog.d/5719.feature

@@ -0,0 +1 @@
+Do not cancel the current incremental sync request and treatment when the app goes to background

matrix-sdk-android/src/main/java/org/matrix/android/sdk/internal/session/sync/job/SyncThread.kt

@@ -107,7 +107,11 @@ internal class SyncThread @Inject constructor(private val syncTask: SyncTask,
             Timber.tag(loggerTag.value).d("Pause sync...")
             isStarted = false
             retryNoNetworkTask?.cancel()
-            syncScope.coroutineContext.cancelChildren()
+            // Do not cancel the current incremental sync.
+            // Incremental sync can be long and it requires the user to wait for the treatment to end,
+            // else all is restarted from the beginning each time the user moves the app to foreground.
+            Timber.tag(loggerTag.value).d("Pause sync... Not cancelling incremental sync")
+            // syncScope.coroutineContext.cancelChildren()
         }
     }