feat: disable profile qr code feature flag (WPB-20230)

Author: sbakhtiarov

app/src/main/kotlin/com/wire/android/di/accountScoped/UserModule.kt

@@ -25,6 +25,7 @@ import com.wire.kalium.logic.feature.asset.DeleteAssetUseCase
 import com.wire.kalium.logic.feature.asset.GetAssetSizeLimitUseCase
 import com.wire.kalium.logic.feature.asset.GetAvatarAssetUseCase
 import com.wire.kalium.logic.feature.client.FinalizeMLSClientAfterE2EIEnrollment
+import com.wire.kalium.logic.feature.client.IsProfileQRCodeDisabledUseCase
 import com.wire.kalium.logic.feature.client.IsWireCellsEnabledForConversationUseCase
 import com.wire.kalium.logic.feature.client.IsWireCellsEnabledUseCase
 import com.wire.kalium.logic.feature.conversation.GetAllContactsNotInConversationUseCase
@@ -254,4 +255,9 @@ class UserModule {
     @Provides
     fun provideIsWireCellsEnabledForConversationUseCase(userScope: UserScope): IsWireCellsEnabledForConversationUseCase =
         userScope.isWireCellsEnabledForConversation
+
+    @ViewModelScoped
+    @Provides
+    fun provideProfileQRCodeConfigUseCase(userScope: UserScope): IsProfileQRCodeDisabledUseCase =
+        userScope.isProfileQRCodeDisabled
 }

app/src/main/kotlin/com/wire/android/ui/debug/featureflags/DebugFeatureFlagsViewModel.kt

@@ -68,6 +68,7 @@ class DebugFeatureFlagsViewModel @Inject constructor(
                             addFeature("Consumable Notifications", consumableNotificationsModel?.status)
                             addFeature("Allowed Global Operations", allowedGlobalOperationsModel?.status, allowedGlobalOperationsModel)
                             addFeature("Wire Cells", cellsModel?.status)
+                            addFeature("Disable Profile QR code", disableUserProfileQRCodeConfigModel?.status)
 
                             add(
                                 Feature(

app/src/main/kotlin/com/wire/android/ui/userprofile/common/UserProfileInfo.kt

@@ -18,7 +18,6 @@
 
 package com.wire.android.ui.userprofile.common
 
-import android.annotation.SuppressLint
 import androidx.compose.animation.AnimatedVisibility
 import androidx.compose.animation.Crossfade
 import androidx.compose.foundation.background
@@ -98,7 +97,7 @@ import kotlin.time.Duration.Companion.hours
 import kotlin.time.Duration.Companion.milliseconds
 import kotlin.time.Duration.Companion.minutes
 
-@SuppressLint("ComposeParameterOrder")
+@Suppress("ComposeParameterOrder", "CyclomaticComplexMethod")
 @Composable
 fun UserProfileInfo(
     userId: UserId?,
@@ -117,7 +116,8 @@ fun UserProfileInfo(
     isMLSVerified: Boolean = false,
     expiresAt: Instant? = null,
     onQrCodeClick: (() -> Unit)? = null,
-    accentId: Int = -1
+    accentId: Int = -1,
+    showQrCode: Boolean = true,
 ) {
     Column(
         horizontalAlignment = CenterHorizontally,
@@ -258,7 +258,7 @@ fun UserProfileInfo(
                         bottom.linkTo(displayName.bottom)
                     }
             ) {
-                if (QRCodeEnabled && isLoading.not()) {
+                if (QRCodeEnabled && showQrCode && isLoading.not()) {
                     onQrCodeClick?.let { QRCodeIcon(it) }
                 }
             }

app/src/main/kotlin/com/wire/android/ui/userprofile/self/SelfUserProfileScreen.kt

@@ -292,7 +292,8 @@ private fun SelfUserProfileContent(
                             onUserProfileClick = onChangeUserProfilePicture,
                             editableState = EditableState.IsEditable(onEditClick),
                             onQrCodeClick = onQrCodeClick,
-                            accentId = accentId
+                            accentId = accentId,
+                            showQrCode = state.showQrCode,
                         )
                     }
                     if (state.legalHoldStatus != LegalHoldUIState.None) {

app/src/main/kotlin/com/wire/android/ui/userprofile/self/SelfUserProfileState.kt

@@ -43,5 +43,6 @@ data class SelfUserProfileState(
     val isAbleToMigrateToTeamAccount: Boolean = false,
     val isLoggingOut: Boolean = false,
     val legalHoldStatus: LegalHoldUIState = LegalHoldUIState.None,
-    val accentId: Int = -1
+    val accentId: Int = -1,
+    val showQrCode: Boolean = false,
 )

app/src/main/kotlin/com/wire/android/ui/userprofile/self/SelfUserProfileViewModel.kt

@@ -53,6 +53,7 @@ import com.wire.kalium.logic.data.user.type.UserType
 import com.wire.kalium.logic.feature.auth.LogoutUseCase
 import com.wire.kalium.logic.feature.call.usecase.EndCallUseCase
 import com.wire.kalium.logic.feature.call.usecase.ObserveEstablishedCallsUseCase
+import com.wire.kalium.logic.feature.client.IsProfileQRCodeDisabledUseCase
 import com.wire.kalium.logic.feature.legalhold.LegalHoldStateForSelfUser
 import com.wire.kalium.logic.feature.legalhold.ObserveLegalHoldStateForSelfUserUseCase
 import com.wire.kalium.logic.feature.personaltoteamaccount.CanMigrateFromPersonalToTeamUseCase
@@ -101,7 +102,8 @@ class SelfUserProfileViewModel @Inject constructor(
     private val globalDataStore: GlobalDataStore,
     private val qualifiedIdMapper: QualifiedIdMapper,
     private val anonymousAnalyticsManager: AnonymousAnalyticsManager,
-    private val getTeamUrl: GetTeamUrlUseCase
+    private val getTeamUrl: GetTeamUrlUseCase,
+    private val isProfileQRCodeDisabled: IsProfileQRCodeDisabledUseCase,
 ) : ViewModel() {
 
     var userProfileState by mutableStateOf(SelfUserProfileState(userId = selfUserId, isAvatarLoading = true))
@@ -116,9 +118,15 @@ class SelfUserProfileViewModel @Inject constructor(
             fetchIsReadOnlyAccount()
             observeLegalHoldStatus()
             markCreateTeamNoticeAsRead()
+            fetchProfileQRCodeState()
         }
     }
 
+    fun fetchProfileQRCodeState() = viewModelScope.launch {
+        val isDisabled = isProfileQRCodeDisabled()
+        userProfileState = userProfileState.copy(showQrCode = !isDisabled)
+    }
+
     suspend fun checkIfUserAbleToMigrateToTeamAccount() {
         val canMigrate = canMigrateFromPersonalToTeam()
         userProfileState = userProfileState.copy(isAbleToMigrateToTeamAccount = canMigrate)

app/src/test/kotlin/com/wire/android/ui/userprofile/self/SelfUserProfileViewModelArrangement.kt

@@ -28,20 +28,21 @@ import com.wire.android.framework.TestUser
 import com.wire.android.mapper.OtherAccountMapper
 import com.wire.android.notification.WireNotificationManager
 import com.wire.android.util.dispatchers.DispatcherProvider
+import com.wire.kalium.common.functional.Either
 import com.wire.kalium.logic.data.id.QualifiedIdMapper
 import com.wire.kalium.logic.feature.auth.LogoutUseCase
 import com.wire.kalium.logic.feature.call.usecase.EndCallUseCase
 import com.wire.kalium.logic.feature.call.usecase.ObserveEstablishedCallsUseCase
+import com.wire.kalium.logic.feature.client.IsProfileQRCodeDisabledUseCase
 import com.wire.kalium.logic.feature.legalhold.LegalHoldStateForSelfUser
 import com.wire.kalium.logic.feature.legalhold.ObserveLegalHoldStateForSelfUserUseCase
 import com.wire.kalium.logic.feature.personaltoteamaccount.CanMigrateFromPersonalToTeamUseCase
 import com.wire.kalium.logic.feature.server.GetTeamUrlUseCase
 import com.wire.kalium.logic.feature.team.GetUpdatedSelfTeamUseCase
-import com.wire.kalium.logic.feature.user.ObserveSelfUserUseCase
 import com.wire.kalium.logic.feature.user.IsReadOnlyAccountUseCase
+import com.wire.kalium.logic.feature.user.ObserveSelfUserUseCase
 import com.wire.kalium.logic.feature.user.ObserveValidAccountsUseCase
 import com.wire.kalium.logic.feature.user.UpdateSelfAvailabilityStatusUseCase
-import com.wire.kalium.common.functional.Either
 import io.mockk.MockKAnnotations
 import io.mockk.coEvery
 import io.mockk.impl.annotations.MockK
@@ -105,6 +106,9 @@ class SelfUserProfileViewModelArrangement {
     @MockK
     lateinit var getTeamUrl: GetTeamUrlUseCase
 
+    @MockK
+    lateinit var profileQRCodeEnabledUseCase: IsProfileQRCodeDisabledUseCase
+
     private val viewModel by lazy {
         SelfUserProfileViewModel(
             selfUserId = TestUser.SELF_USER.id,
@@ -126,7 +130,8 @@ class SelfUserProfileViewModelArrangement {
             qualifiedIdMapper = qualifiedIdMapper,
             anonymousAnalyticsManager = anonymousAnalyticsManager,
             canMigrateFromPersonalToTeam = canMigrateFromPersonalToTeam,
-            getTeamUrl = getTeamUrl
+            getTeamUrl = getTeamUrl,
+            isProfileQRCodeDisabled = profileQRCodeEnabledUseCase,
         )
     }
 
@@ -142,6 +147,7 @@ class SelfUserProfileViewModelArrangement {
         coEvery { observeEstablishedCalls.invoke() } returns flowOf(emptyList())
         coEvery { canMigrateFromPersonalToTeam.invoke() } returns true
         coEvery { getTeamUrl.invoke() } returns ""
+        coEvery { profileQRCodeEnabledUseCase.invoke() } returns true
     }
 
     fun withLegalHoldStatus(result: LegalHoldStateForSelfUser) = apply {