ehealthsolutions/element-x-ios
3
0
Fork 0
mirror of https://github.com/Novaloop-AG/element-x-ios.git synced 2026-11-02 12:07:55 +00:00
Code Issues Packages Projects Releases Wiki Activity

Automatically dismiss UserDetailsEditScreen when saving changes. (#5073)

Browse Source
This commit is contained in:
Doug
2026-02-09 15:07:50 +00:00
committed by GitHub
parent e23685be70
commit 14845d5620
2 changed files with 4 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files
ElementX/Sources/Screens/Settings/UserDetailsEditScreen/UserDetailsEditScreenViewModel.swift
+3 -5
View File
@@ -112,11 +112,11 @@ class UserDetailsEditScreenViewModel: UserDetailsEditScreenViewModelType, UserDe
state.bindings.alertInfo = .init(id: .unsavedChanges,
title: L10n.dialogUnsavedChangesTitle,
message: L10n.dialogUnsavedChangesDescription,
primaryButton: .init(title: L10n.actionSave) { Task { await self.saveUserDetails(shouldDismiss: true) } },
primaryButton: .init(title: L10n.actionSave) { Task { await self.saveUserDetails() } },
secondaryButton: .init(title: L10n.actionDiscard, role: .cancel) { self.actionsSubject.send(.dismiss) })
}
private func saveUserDetails(shouldDismiss: Bool = false) async {
private func saveUserDetails() async {
let userIndicatorID = UUID().uuidString
defer {
userIndicatorController.retractIndicatorWithId(userIndicatorID)
@@ -147,9 +147,7 @@ class UserDetailsEditScreenViewModel: UserDetailsEditScreenViewModelType, UserDe
try await group.waitForAll()
}
if shouldDismiss {
actionsSubject.send(.dismiss)
}
actionsSubject.send(.dismiss)
} catch {
userIndicatorController.alertInfo = .init(id: .init(),
title: L10n.screenEditProfileErrorTitle,
UnitTests/Sources/UserDetailsEditScreenViewModelTests.swift
+1 -2
View File
@@ -49,8 +49,7 @@ class UserDetailsEditScreenViewModelTests: XCTestCase {
func testSave() async throws {
setupViewModel()
// Saving shouldn't dismiss this screen (or trigger any other action).
let deferred = deferFailure(viewModel.actions, timeout: 1) { _ in true }
let deferred = deferFulfillment(viewModel.actions) { $0 == .dismiss }
context.name = "name"
context.send(viewAction: .save)