From 9d1d5142da68a4511cbd022863bdfb4d9bbf6b57 Mon Sep 17 00:00:00 2001 From: Michelle Tang Date: Wed, 20 May 2026 10:14:14 -0400 Subject: [PATCH] Turn on key transparency. --- .../AdvancedPrivacySettingsFragment.kt | 43 +++++++++---------- .../securesms/jobs/CheckKeyTransparencyJob.kt | 5 +-- .../verify/VerifyDisplayScreenState.kt | 5 +-- 3 files changed, 23 insertions(+), 30 deletions(-) diff --git a/app/src/main/java/org/thoughtcrime/securesms/components/settings/app/privacy/advanced/AdvancedPrivacySettingsFragment.kt b/app/src/main/java/org/thoughtcrime/securesms/components/settings/app/privacy/advanced/AdvancedPrivacySettingsFragment.kt index 45b938ba71..283c243dbf 100644 --- a/app/src/main/java/org/thoughtcrime/securesms/components/settings/app/privacy/advanced/AdvancedPrivacySettingsFragment.kt +++ b/app/src/main/java/org/thoughtcrime/securesms/components/settings/app/privacy/advanced/AdvancedPrivacySettingsFragment.kt @@ -45,7 +45,6 @@ import org.signal.core.ui.compose.Texts import org.thoughtcrime.securesms.R import org.thoughtcrime.securesms.compose.rememberStatusBarColorNestedScrollModifier import org.thoughtcrime.securesms.util.CommunicationActions -import org.thoughtcrime.securesms.util.RemoteConfig import org.thoughtcrime.securesms.util.viewModel /** @@ -299,31 +298,29 @@ private fun AdvancedPrivacySettingsScreen( ) } - if (RemoteConfig.internalUser) { - item { - Dividers.Default() - } + item { + Dividers.Default() + } - item { - val label = buildAnnotatedString { - append(stringResource(R.string.preferences_automatic_key_verification_body)) - append(" ") - withLink( - LinkAnnotation.Clickable("learn-more", linkInteractionListener = { - callbacks.onAutomaticVerificationLearnMoreClick() - }) - ) { - append(stringResource(R.string.LearnMoreTextView_learn_more)) - } + item { + val label = buildAnnotatedString { + append(stringResource(R.string.preferences_automatic_key_verification_body)) + append(" ") + withLink( + LinkAnnotation.Clickable("learn-more", linkInteractionListener = { + callbacks.onAutomaticVerificationLearnMoreClick() + }) + ) { + append(stringResource(R.string.LearnMoreTextView_learn_more)) } - - Rows.ToggleRow( - checked = state.allowAutomaticKeyVerification, - text = AnnotatedString(stringResource(R.string.preferences_automatic_key_verification)), - label = label, - onCheckChanged = callbacks::onAllowAutomaticVerificationChanged - ) } + + Rows.ToggleRow( + checked = state.allowAutomaticKeyVerification, + text = AnnotatedString(stringResource(R.string.preferences_automatic_key_verification)), + label = label, + onCheckChanged = callbacks::onAllowAutomaticVerificationChanged + ) } } } diff --git a/app/src/main/java/org/thoughtcrime/securesms/jobs/CheckKeyTransparencyJob.kt b/app/src/main/java/org/thoughtcrime/securesms/jobs/CheckKeyTransparencyJob.kt index 78998695e0..db4c040e1e 100644 --- a/app/src/main/java/org/thoughtcrime/securesms/jobs/CheckKeyTransparencyJob.kt +++ b/app/src/main/java/org/thoughtcrime/securesms/jobs/CheckKeyTransparencyJob.kt @@ -81,10 +81,7 @@ class CheckKeyTransparencyJob private constructor( } private fun canRunJob(): Boolean { - return if (!RemoteConfig.internalUser) { - Log.i(TAG, "Remote config is not on. Exiting.") - false - } else if (!SignalStore.account.isRegistered) { + return if (!SignalStore.account.isRegistered) { Log.i(TAG, "Account not registered. Exiting.") false } else if (!SignalStore.settings.automaticVerificationEnabled) { diff --git a/app/src/main/java/org/thoughtcrime/securesms/verify/VerifyDisplayScreenState.kt b/app/src/main/java/org/thoughtcrime/securesms/verify/VerifyDisplayScreenState.kt index 1b709a7855..cce024ba3c 100644 --- a/app/src/main/java/org/thoughtcrime/securesms/verify/VerifyDisplayScreenState.kt +++ b/app/src/main/java/org/thoughtcrime/securesms/verify/VerifyDisplayScreenState.kt @@ -7,12 +7,11 @@ package org.thoughtcrime.securesms.verify import org.thoughtcrime.securesms.keyvalue.SignalStore import org.thoughtcrime.securesms.recipients.Recipient -import org.thoughtcrime.securesms.util.RemoteConfig data class VerifyDisplayScreenState( val isSafetyNumberVerified: Boolean, - val isAutomaticVerificationVisible: Boolean = RemoteConfig.internalUser && SignalStore.settings.automaticVerificationEnabled, - val shouldDisplayVerifyAutomaticallyEducationSheet: Boolean = RemoteConfig.internalUser && SignalStore.settings.automaticVerificationEnabled && !SignalStore.uiHints.hasSeenVerifyAutomaticallySheet(), + val isAutomaticVerificationVisible: Boolean = SignalStore.settings.automaticVerificationEnabled, + val shouldDisplayVerifyAutomaticallyEducationSheet: Boolean = SignalStore.settings.automaticVerificationEnabled && !SignalStore.uiHints.hasSeenVerifyAutomaticallySheet(), val recipient: Recipient? = null, val fingerprintHolder: FingerprintHolder = FingerprintHolder.Uninitialised, val automaticVerificationStatus: AutomaticVerificationStatus = AutomaticVerificationStatus.NONE,