self review

wireapp · Nov 13, 2024 · f60103a · f60103a
1 parent 31265e6
commit f60103a
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 5 deletions.
diff --git a/logic/src/commonMain/kotlin/com/wire/kalium/logic/data/user/UserRepository.kt b/logic/src/commonMain/kotlin/com/wire/kalium/logic/data/user/UserRepository.kt
@@ -733,7 +733,7 @@ internal class UserDataSource internal constructor(
             .onSuccess {
                 kaliumLogger.d("Migrated user to team")
                 fetchSelfUser()
-                // TODO Invalidate team id in memory so UserSessionScope.selfTeamId got updated data
+                // TODO Invalidate team id in memory so UserSessionScope.selfTeamId got updated data WPB-12187
             }
             .onFailure { failure ->
                 kaliumLogger.e("Failed to migrate user to team: $failure")

diff --git a/...k/src/commonMain/kotlin/com/wire/kalium/network/api/v0/authenticated/ConversationApiV0.kt b/...k/src/commonMain/kotlin/com/wire/kalium/network/api/v0/authenticated/ConversationApiV0.kt
@@ -395,6 +395,7 @@ internal open class ConversationApiV0 internal constructor(
             httpClient.get("$PATH_CONVERSATIONS/${conversationId.value}/$PATH_CODE")
         }
 
+
     protected companion object {
         const val PATH_CONVERSATIONS = "conversations"
         const val PATH_SELF = "self"

diff --git a/...monMain/kotlin/com/wire/kalium/network/api/v7/authenticated/UpgradePersonalToTeamApiV7.kt b/...monMain/kotlin/com/wire/kalium/network/api/v7/authenticated/UpgradePersonalToTeamApiV7.kt
@@ -41,7 +41,7 @@ internal open class UpgradePersonalToTeamApiV7 internal constructor(
                         name = teamName,
                         iconAssetId = "default",
                         iconKey = "abc",
-                        currency = null,//TODO is this ok?
+                        currency = null,
                     )
                 )
             }

diff --git a/...mmonMain/kotlin/com/wire/kalium/network/networkContainer/AuthenticatedNetworkContainer.kt b/...mmonMain/kotlin/com/wire/kalium/network/networkContainer/AuthenticatedNetworkContainer.kt
@@ -49,6 +49,7 @@ import com.wire.kalium.network.api.v0.authenticated.networkContainer.Authenticat
 import com.wire.kalium.network.api.v2.authenticated.networkContainer.AuthenticatedNetworkContainerV2
 import com.wire.kalium.network.api.v4.authenticated.networkContainer.AuthenticatedNetworkContainerV4
 import com.wire.kalium.network.api.v5.authenticated.networkContainer.AuthenticatedNetworkContainerV5
+import com.wire.kalium.network.api.v6.authenticated.networkContainer.AuthenticatedNetworkContainerV6
 import com.wire.kalium.network.api.v7.authenticated.networkContainer.AuthenticatedNetworkContainerV7
 import com.wire.kalium.network.session.CertificatePinning
 import com.wire.kalium.network.session.SessionManager
@@ -182,9 +183,7 @@ interface AuthenticatedNetworkContainer {
                     kaliumLogger
                 )
 
-
-                //TODO revert object to v6
-                6 -> AuthenticatedNetworkContainerV7(
+                6 -> AuthenticatedNetworkContainerV6(
                     sessionManager,
                     selfUserId,
                     certificatePinning,