fix(agent): 🐛 actually save agent preferences and show better dialogs…

… for success/fail
joshuar · Jul 26, 2024 · dfd1c47 · dfd1c47
1 parent bf1f6c4
commit dfd1c47
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/internal/agent/ui/fyneUI/fyneUI.go b/internal/agent/ui/fyneUI/fyneUI.go
@@ -255,8 +255,13 @@ func (i *FyneUI) agentSettingsWindow(ctx context.Context) fyne.Window {
 		prefs.MQTTUser = mqttPrefs.User
 		prefs.MQTTPassword = mqttPrefs.Password
 
-		dialog.ShowInformation("Saved", "MQTT Preferences have been saved.", window)
-		logging.FromContext(ctx).Info("MQTT preferences saved.")
+		if err := prefs.Save(); err != nil {
+			dialog.ShowError(err, window)
+			logging.FromContext(ctx).Error("Failed to save MQTT preferences.", "error", err.Error())
+		} else {
+			dialog.ShowInformation("Saved", "MQTT Preferences have been saved. Restart agent to utilise them.", window)
+			logging.FromContext(ctx).Info("Saved MQTT preferences.")
+		}
 	}
 	settingsForm.OnCancel = func() {
 		window.Close()