From f14cb45680048790d0073053ccd2456ebf7d2ca4 Mon Sep 17 00:00:00 2001
From: Tobias Gesellchen <tobias@gesellix.de>
Date: Fri, 17 Apr 2026 21:51:11 +0200
Subject: [PATCH] Enhance and group device discovery settings in web UI (#169)

---
 pkg/service/handlers/handlers_setup.go |  1 -
 pkg/service/handlers/web/index.html    | 13 ++++++++++---
 pkg/service/handlers/web/js/script.js  |  8 +++++++-
 3 files changed, 17 insertions(+), 5 deletions(-)

diff --git a/pkg/service/handlers/handlers_setup.go b/pkg/service/handlers/handlers_setup.go
index 74d5990..f637913 100644
--- a/pkg/service/handlers/handlers_setup.go
+++ b/pkg/service/handlers/handlers_setup.go
@@ -86,7 +86,6 @@ func (s *Server) HandleTriggerDiscovery(w http.ResponseWriter, _ *http.Request)
 	go s.DiscoverDevices(context.Background())
 
 	w.WriteHeader(http.StatusAccepted)
-	_, _ = w.Write([]byte(`{"status": "Discovery started"}`))
 }
 
 // HandleGetDiscoveryStatus returns the current discovery status.
diff --git a/pkg/service/handlers/web/index.html b/pkg/service/handlers/web/index.html
index 06c070d..4493b28 100644
--- a/pkg/service/handlers/web/index.html
+++ b/pkg/service/handlers/web/index.html
@@ -152,9 +152,16 @@
             <span style="font-size: 0.8em; color: #666">(Standard services URL)</span>
         </div>
         <div style="margin-bottom: 20px">
-            <label for="discovery-interval">Discovery Interval:</label>
-            <input type="text" id="discovery-interval" placeholder="5m" style="width: 100px"/>
-            <label style="margin-left: 15px"><input type="checkbox" id="discovery-enabled"/> Enable Automated Discovery</label>
+            <strong>Device Discovery:</strong>
+            <div style="margin-top: 5px">
+                <label style="display: block; margin-bottom: 5px">
+                    <input type="checkbox" id="discovery-enabled"/> Enable Automated Discovery
+                </label>
+                <div style="margin-left: 20px">
+                    <label for="discovery-interval">Discovery Interval:</label>
+                    <input type="text" id="discovery-interval" placeholder="5m" style="width: 100px"/>
+                </div>
+            </div>
         </div>
         <div style="margin-bottom: 20px">
             <strong>DNS Discovery:</strong>
diff --git a/pkg/service/handlers/web/js/script.js b/pkg/service/handlers/web/js/script.js
index a06f56f..4279849 100644
--- a/pkg/service/handlers/web/js/script.js
+++ b/pkg/service/handlers/web/js/script.js
@@ -1535,7 +1535,13 @@ async function triggerDiscovery() {
     const indicator = document.getElementById("discovery-indicator");
     if (indicator) indicator.style.display = "inline";
     try {
-        await fetch("/setup/discover", {method: "POST"});
+        const response = await fetch("/setup/discover", {method: "POST"});
+        if (!response.ok) {
+            const err = await response.text();
+            alert("Failed to start discovery: " + err);
+            if (indicator) indicator.style.display = "none";
+            return;
+        }
         pollDiscoveryStatus();
     } catch (error) {
         console.error("Failed to trigger discovery", error);