Add comment to remember the problem with the connectivity-status library

This commit is contained in:
davidoskky 2022-08-20 21:28:21 +02:00
parent 767805981f
commit 9d0bb452e3
2 changed files with 6 additions and 0 deletions

View File

@ -203,5 +203,6 @@ dependencies {
implementation("android.arch.work:work-runtime-ktx:1.0.1") implementation("android.arch.work:work-runtime-ktx:1.0.1")
// Network information // Network information
// TODO: When updating this library, check if the todo in RepositoryImpl.startNetwork can be resolved
implementation("com.github.ln-12:multiplatform-connectivity-status:1.1.0") implementation("com.github.ln-12:multiplatform-connectivity-status:1.1.0")
} }

View File

@ -341,6 +341,11 @@ class Repository(private val api: SelfossApi, private val apiDetails: ApiDetails
fun isNetworkAvailable() = isConnectionAvailable.value && !offlineOverride fun isNetworkAvailable() = isConnectionAvailable.value && !offlineOverride
// TODO: Start and stop the monitoring within Android and iOS code
// This should be performed in the architecture specific code
// However it is not possible on android due to the dash contained in the name of the library
// com.github.ln-12:multiplatform-connectivity-status:1.1.0
// https://github.com/ln-12/multiplatform-connectivity-status/issues/2
fun startNetwork() { fun startNetwork() {
connectivityStatus.start() connectivityStatus.start()
} }