Skip to content

Commit 38a022c

Browse files
author
Darin Krauss
authored
[LOOP-3913] Fix Critical Event Log Export failure (#479)
- https://tidepool.atlassian.net/browse/LOOP-3913 - Ensure export is invoked once only
1 parent 6e1ef64 commit 38a022c

File tree

2 files changed

+8
-1
lines changed

2 files changed

+8
-1
lines changed

Loop/View Models/CriticalEventLogExportViewModel.swift

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@ protocol CriticalEventLogExporterFactory {
1919
extension CriticalEventLogExportManager: CriticalEventLogExporterFactory {}
2020

2121
public class CriticalEventLogExportViewModel: ObservableObject, Identifiable, CriticalEventLogExporterDelegate {
22+
@Published var isExporting: Bool = false
2223
@Published var showingSuccess: Bool = false
2324
@Published var showingShare: Bool = false
2425
@Published var showingError: Bool = false
@@ -83,6 +84,12 @@ public class CriticalEventLogExportViewModel: ObservableObject, Identifiable, Cr
8384
func export() {
8485
dispatchPrecondition(condition: .onQueue(.main))
8586

87+
guard !isExporting else {
88+
return
89+
}
90+
91+
self.isExporting = true
92+
8693
self.showingSuccess = false
8794
self.showingShare = false
8895
self.showingError = false
@@ -126,6 +133,7 @@ public class CriticalEventLogExportViewModel: ObservableObject, Identifiable, Cr
126133
self.exporter?.cancel()
127134
self.exporter = nil
128135
self.activityItems = []
136+
self.isExporting = false
129137
}
130138

131139
// MARK: - CriticalEventLogExporterDelegate

Loop/Views/CriticalEventLogExportView.swift

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,6 @@ struct CriticalEventLogExportView: View {
3030
.navigationBarBackButtonHidden(true)
3131
.navigationBarItems(leading: cancelButton)
3232
.onAppear { self.viewModel.export() }
33-
.onDisappear { self.viewModel.cancel() }
3433
.alert(isPresented: $viewModel.showingError) {
3534
errorAlert
3635
}

0 commit comments

Comments
 (0)