Skip to content
Snippets Groups Projects
Commit d407039f authored by Bruno Muniz's avatar Bruno Muniz :apple:
Browse files

Fix comments on MR

parent 93518f3d
No related branches found
No related tags found
2 merge requests!71Releasing v1.1.5 (214),!70Add user defaults flag for reporting enabled and reporting optional
......@@ -63,7 +63,8 @@ public final class SettingsAdvancedController: UIViewController {
screenView.reportingSwitcher.switcherView
.publisher(for: .valueChanged)
.sink { [weak viewModel] in viewModel?.didToggleReporting() }
.compactMap { [weak screenView] _ in screenView?.reportingSwitcher.switcherView.isOn }
.sink { [weak viewModel] isOn in viewModel?.didSetReporting(enabled: isOn) }
.store(in: &cancellables)
viewModel.sharePublisher
......
......@@ -18,6 +18,7 @@ final class SettingsAdvancedViewModel {
@KeyObject(.recordingLogs, defaultValue: true) var isRecordingLogs: Bool
@KeyObject(.crashReporting, defaultValue: true) var isCrashReporting: Bool
private var cancellables = Set<AnyCancellable>()
private let isShowingUsernamesKey = "isShowingUsernames"
@Dependency private var logger: XXLogger
......@@ -33,9 +34,13 @@ final class SettingsAdvancedViewModel {
func loadCachedSettings() {
stateRelay.value.isRecordingLogs = isRecordingLogs
stateRelay.value.isCrashReporting = isCrashReporting
stateRelay.value.isReportingEnabled = reportingStatus.isEnabled()
stateRelay.value.isReportingOptional = reportingStatus.isOptional()
reportingStatus
.isEnabledPublisher()
.sink { [weak stateRelay] in stateRelay?.value.isReportingEnabled = $0 }
.store(in: &cancellables)
guard let defaults = UserDefaults(suiteName: "group.elixxir.messenger") else {
print("^^^ Couldn't access user defaults in the app group container \(#file):\(#line)")
return
......@@ -82,8 +87,7 @@ final class SettingsAdvancedViewModel {
crashReporter.setEnabled(isCrashReporting)
}
func didToggleReporting() {
reportingStatus.enable(reportingStatus.isEnabled())
stateRelay.value.isReportingEnabled.toggle()
func didSetReporting(enabled: Bool) {
reportingStatus.enable(enabled)
}
}
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment