Enable the App Lock feature.

ElementX/Sources/Application/AppCoordinator.swift

@@ -149,9 +149,7 @@ class AppCoordinator: AppCoordinatorProtocol, AuthenticationCoordinatorDelegate,
             return
         }
 
-        if appSettings.appLockFlowEnabled,
-           appSettings.appLockIsMandatory,
-           !appLockFlowCoordinator.appLockService.isEnabled {
+        if appSettings.appLockIsMandatory, !appLockFlowCoordinator.appLockService.isEnabled {
             stateMachine.processEvent(.startWithAppLockSetup)
         } else {
             stateMachine.processEvent(.startWithExistingSession)

ElementX/Sources/Application/AppSettings.swift

@@ -42,7 +42,6 @@ final class AppSettings {
         case userSuggestionsEnabled
         case readReceiptsEnabled
         case swiftUITimelineEnabled
-        case appLockFlowEnabled
         case elementCallEnabled
         case chatBackupEnabled
     }
@@ -268,9 +267,6 @@ final class AppSettings {
     @UserPreference(key: UserDefaultsKeys.swiftUITimelineEnabled, defaultValue: false, storageType: .volatile)
     var swiftUITimelineEnabled
 
-    @UserPreference(key: UserDefaultsKeys.appLockFlowEnabled, defaultValue: false, storageType: .volatile)
-    var appLockFlowEnabled
-
     @UserPreference(key: UserDefaultsKeys.elementCallEnabled, defaultValue: true, storageType: .userDefaults(store))
     var elementCallEnabled
 

ElementX/Sources/Mocks/Generated/GeneratedMocks.swift

@@ -1,4 +1,4 @@
-// Generated using Sourcery 2.1.1 — https://github.com/krzysztofzablocki/Sourcery
+// Generated using Sourcery 2.1.2 — https://github.com/krzysztofzablocki/Sourcery
 // DO NOT EDIT
 
 // swiftlint:disable all

ElementX/Sources/Screens/Settings/DeveloperOptionsScreen/DeveloperOptionsScreenModels.swift

@@ -49,7 +49,6 @@ protocol DeveloperOptionsProtocol: AnyObject {
     var userSuggestionsEnabled: Bool { get set }
     var readReceiptsEnabled: Bool { get set }
     var swiftUITimelineEnabled: Bool { get set }
-    var appLockFlowEnabled: Bool { get set }
     var elementCallEnabled: Bool { get set }
     var chatBackupEnabled: Bool { get set }
 }

ElementX/Sources/Screens/Settings/DeveloperOptionsScreen/View/DeveloperOptionsScreen.swift

@@ -36,11 +36,6 @@ struct DeveloperOptionsScreen: View {
                     Text("Chat backup")
                     Text("Requires app reboot")
                 }
-
-                Toggle(isOn: $context.appLockFlowEnabled) {
-                    Text("PIN/Biometric lock")
-                    Text("Resets on reboot")
-                }
             }
 
             Section("Timeline") {

ElementX/Sources/Screens/Settings/SettingsScreen/SettingsScreenModels.swift

@@ -44,7 +44,6 @@ struct SettingsScreenViewState: BindableState {
     var isSessionVerified = false
     var chatBackupEnabled = false
     var showSecureBackupBadge = false
-    var showAppLockSettings: Bool
     var showDeveloperOptions: Bool
 
     /// The presentation anchor used to display the OIDC account URL.

ElementX/Sources/Screens/Settings/SettingsScreen/SettingsScreenViewModel.swift

@@ -37,7 +37,6 @@ class SettingsScreenViewModel: SettingsScreenViewModelType, SettingsScreenViewMo
                                            userID: userSession.userID,
                                            accountProfileURL: userSession.clientProxy.accountURL(action: .profile),
                                            accountSessionsListURL: userSession.clientProxy.accountURL(action: .sessionsList),
-                                           showAppLockSettings: appSettings.appLockFlowEnabled,
                                            showDeveloperOptions: appSettings.canShowDeveloperOptions),
                    imageProvider: userSession.mediaProvider)
 
@@ -50,10 +49,6 @@ class SettingsScreenViewModel: SettingsScreenViewModelType, SettingsScreenViewMo
             .receive(on: DispatchQueue.main)
             .weakAssign(to: \.state.userDisplayName, on: self)
             .store(in: &cancellables)
-
-        appSettings.$appLockFlowEnabled
-            .weakAssign(to: \.state.showAppLockSettings, on: self)
-            .store(in: &cancellables)
 
         appSettings.$chatBackupEnabled
             .weakAssign(to: \.state.chatBackupEnabled, on: self)

ElementX/Sources/Screens/Settings/SettingsScreen/View/SettingsScreen.swift

@@ -125,14 +125,12 @@ struct SettingsScreen: View {
                     })
                     .accessibilityIdentifier(A11yIdentifiers.settingsScreen.analytics)
 
-            if context.viewState.showAppLockSettings {
-                ListRow(label: .default(title: L10n.commonScreenLock,
-                                        systemIcon: .lock),
-                        kind: .navigationLink {
-                            context.send(viewAction: .appLock)
-                        })
-                        .accessibilityIdentifier(A11yIdentifiers.settingsScreen.screenLock)
-            }
+            ListRow(label: .default(title: L10n.commonScreenLock,
+                                    systemIcon: .lock),
+                    kind: .navigationLink {
+                        context.send(viewAction: .appLock)
+                    })
+                    .accessibilityIdentifier(A11yIdentifiers.settingsScreen.screenLock)
 
             ListRow(label: .default(title: L10n.commonReportABug,
                                     systemIcon: .ladybug),

ElementX/Sources/Services/AppLock/AppLockService.swift

@@ -30,7 +30,6 @@ class AppLockService: AppLockServiceProtocol {
 
     var isEnabled: Bool {
         do {
-            guard appSettings.appLockFlowEnabled else { return false }
             return try keychainController.containsPINCode()
         } catch {
             MXLog.error("Keychain access error: \(error)")

UnitTests/Sources/AppLock/AppLockServiceTests.swift

@@ -28,7 +28,6 @@ class AppLockServiceTests: XCTestCase {
     override func setUp() {
         AppSettings.reset()
         appSettings = AppSettings()
-        appSettings.appLockFlowEnabled = true
 
         keychainController = KeychainController(service: .tests, accessGroup: InfoPlistReader.main.keychainAccessGroupIdentifier)
         keychainController.resetSecrets()

UnitTests/Sources/AppLock/AppLockSettingsScreenViewModelTests.swift

@@ -29,19 +29,11 @@ class AppLockSetupSettingsScreenViewModelTests: XCTestCase {
     }
 
     override func setUpWithError() throws {
-        AppSettings.reset()
-        let appSettings = AppSettings()
-        appSettings.appLockFlowEnabled = true
-
         keychainController = KeychainControllerMock()
-        appLockService = AppLockService(keychainController: keychainController, appSettings: appSettings)
+        appLockService = AppLockService(keychainController: keychainController, appSettings: AppSettings())
 
         viewModel = AppLockSetupSettingsScreenViewModel(appLockService: AppLockServiceMock.mock())
     }
-
-    override func tearDown() {
-        AppSettings.reset()
-    }
 
     func testDisablingShowsAlert() {
         // Given a fresh screen with the PIN code enabled.

changelog.d/pr-1990.feature

@@ -0,0 +1 @@
+Allow the app to be locked with a PIN code or Touch/Face ID.