Merge pull request #580 from DroidKaigi/fix-firebase-dependencies

Changed to not include Firebase implementation on the iOS side

core/data/build.gradle.kts

@@ -24,7 +24,6 @@ kotlin {
                 implementation(libs.ktorKotlinxSerialization)
                 implementation(libs.ktorContentNegotiation)
                 implementation(libs.kermit)
-                implementation(libs.firebaseRemoteConfig)
             }
         }
         val androidMain by getting {
@@ -33,6 +32,7 @@ kotlin {
                 implementation(libs.ktorClientOkHttp)
                 implementation(libs.okHttpLoggingInterceptor)
                 implementation(libs.okHttpLoggingInterceptor)
+                implementation(libs.firebaseRemoteConfig)
             }
         }
         val iosMain by getting {

core/data/src/androidMain/kotlin/io/github/droidkaigi/confsched2023/data/remoteconfig/DefaultRemoteConfigApi.kt

@@ -0,0 +1,26 @@
+package io.github.droidkaigi.confsched2023.data.remoteconfig
+
+import dev.gitlive.firebase.Firebase
+import dev.gitlive.firebase.remoteconfig.get
+import dev.gitlive.firebase.remoteconfig.remoteConfig
+
+actual class DefaultRemoteConfigApi : RemoteConfigApi {
+
+    private val firebaseRemoteConfig = Firebase.remoteConfig
+
+    /**
+     * If you want to change the interval time to fetch, please change it here
+     */
+//    init {
+//        CoroutineScope(Dispatchers.IO).launch {
+//            firebaseRemoteConfig.settings {
+//                minimumFetchIntervalInSeconds = 12 * 3600L
+//            }
+//        }
+//    }
+
+    actual override suspend fun getBoolean(key: String): Boolean {
+        firebaseRemoteConfig.fetchAndActivate()
+        return firebaseRemoteConfig[key]
+    }
+}

core/data/src/commonMain/kotlin/io/github/droidkaigi/confsched2023/data/remoteconfig/DefaultRemoteConfigApi.kt

@@ -1,26 +1,5 @@
 package io.github.droidkaigi.confsched2023.data.remoteconfig
 
-import dev.gitlive.firebase.Firebase
-import dev.gitlive.firebase.remoteconfig.get
-import dev.gitlive.firebase.remoteconfig.remoteConfig
-
-class DefaultRemoteConfigApi : RemoteConfigApi {
-
-    private val firebaseRemoteConfig = Firebase.remoteConfig
-
-    /**
-     * If you want to change the interval time to fetch, please change it here
-     */
-//    init {
-//        CoroutineScope(Dispatchers.IO).launch {
-//            firebaseRemoteConfig.settings {
-//                minimumFetchIntervalInSeconds = 12 * 3600L
-//            }
-//        }
-//    }
-
-    override suspend fun getBoolean(key: String): Boolean {
-        firebaseRemoteConfig.fetchAndActivate()
-        return firebaseRemoteConfig[key]
-    }
+expect class DefaultRemoteConfigApi : RemoteConfigApi {
+    override suspend fun getBoolean(key: String): Boolean
 }

core/data/src/commonMain/kotlin/io/github/droidkaigi/confsched2023/data/remoteconfig/FakeRemoteConfigApi.kt

@@ -25,6 +25,6 @@ class FakeRemoteConfigApi : RemoteConfigApi {
     }
 
     override suspend fun getBoolean(key: String): Boolean {
-        return status.getBoolean("key")
+        return status.getBoolean(key)
     }
 }

core/data/src/iosMain/kotlin/io/github/droidkaigi/confsched2023/data/remoteconfig/DefaultRemoteConfigApi.kt

@@ -0,0 +1,12 @@
+package io.github.droidkaigi.confsched2023.data.remoteconfig
+
+import io.github.droidkaigi.confsched2023.data.remoteconfig.FakeRemoteConfigApi.Status.Default
+
+actual class DefaultRemoteConfigApi : RemoteConfigApi {
+
+    actual override suspend fun getBoolean(key: String): Boolean {
+        return FakeRemoteConfigApi().apply {
+            setUp(Default)
+        }.getBoolean(key)
+    }
+}