Quicker auto-discovery

Immediately show server chooser after the first server has been discovered, replace withContext() with flowOn()
jellyfin · Sep 20, 2020 · bc6ea9b · bc6ea9b
1 parent df24a1d
commit bc6ea9b
Showing 1 changed file with 4 additions and 7 deletions.
diff --git a/app/src/main/java/org/jellyfin/mobile/webapp/ConnectionHelper.kt b/app/src/main/java/org/jellyfin/mobile/webapp/ConnectionHelper.kt
@@ -18,9 +18,8 @@ import androidx.core.view.postDelayed
 import androidx.lifecycle.lifecycleScope
 import kotlinx.coroutines.Dispatchers
 import kotlinx.coroutines.flow.collect
-import kotlinx.coroutines.flow.onEach
+import kotlinx.coroutines.flow.flowOn
 import kotlinx.coroutines.launch
-import kotlinx.coroutines.withContext
 import okhttp3.HttpUrl
 import okhttp3.HttpUrl.Companion.toHttpUrlOrNull
 import org.jellyfin.apiclient.Jellyfin
@@ -147,12 +146,10 @@ class ConnectionHelper(private val activity: MainActivity) : KoinComponent {
 
     private fun discoverServers() {
         activity.lifecycleScope.launch {
-            withContext(Dispatchers.IO) {
-                jellyfin.discovery.discover().onEach {
-                    serverList.add(it)
-                }.collect()
+            jellyfin.discovery.discover().flowOn(Dispatchers.IO).collect { serverInfo ->
+                serverList.add(serverInfo)
+                chooseServerButton.isVisible = true
             }
-            chooseServerButton.isVisible = serverList.isNotEmpty()
         }
     }