From 319ea2d01d39f37381bcff814859f2e4bd383d34 Mon Sep 17 00:00:00 2001
From: Smittix <jamessmith1@outlook.com>
Date: Thu, 15 Jan 2026 13:13:33 +0000
Subject: [PATCH] Remove redundant SDR device selector from APRS configuration

APRS now uses the global device selector in the sidebar like other modes
---
 templates/index.html               | 27 +--------------------------
 templates/partials/modes/aprs.html |  6 ------
 2 files changed, 1 insertion(+), 32 deletions(-)

diff --git a/templates/index.html b/templates/index.html
index 9b34b33..8191736 100644
--- a/templates/index.html
+++ b/templates/index.html
@@ -2092,7 +2092,6 @@
             } else if (mode === 'aprs') {
                 checkAprsTools();
                 initAprsMap();
-                refreshAprsDevices();
                 // Fix map sizing on mobile after container becomes visible
                 setTimeout(() => {
                     if (aprsMap) aprsMap.invalidateSize();
@@ -7309,30 +7308,6 @@
             }, 1000);
         }
 
-        function refreshAprsDevices() {
-            fetch('/devices')
-                .then(r => r.json())
-                .then(devices => {
-                    const select = document.getElementById('aprsDevice');
-                    select.innerHTML = '';
-                    if (devices.length === 0) {
-                        select.innerHTML = '<option value="">No SDR devices found</option>';
-                    } else {
-                        devices.forEach(dev => {
-                            const opt = document.createElement('option');
-                            opt.value = dev.index;
-                            opt.textContent = `${dev.index}: ${dev.name || 'Unknown'}`;
-                            select.appendChild(opt);
-                        });
-                    }
-                })
-                .catch(err => {
-                    console.error('Failed to refresh APRS devices:', err);
-                    const select = document.getElementById('aprsDevice');
-                    select.innerHTML = '<option value="0">Error loading devices</option>';
-                });
-        }
-
         function updateAprsStatus(state, freq) {
             const statusBar = document.getElementById('aprsStatusBar');
             const statusDot = document.getElementById('aprsStatusDot');
@@ -7361,7 +7336,7 @@
 
         function startAprs() {
             const region = document.getElementById('aprsRegion').value;
-            const device = document.getElementById('aprsDevice').value;
+            const device = getSelectedDevice();
             const gain = document.getElementById('aprsGain').value;
 
             fetch('/aprs/start', {
diff --git a/templates/partials/modes/aprs.html b/templates/partials/modes/aprs.html
index 6a8ea6f..b027fde 100644
--- a/templates/partials/modes/aprs.html
+++ b/templates/partials/modes/aprs.html
@@ -21,12 +21,6 @@
                                 <option value="japan">Japan (144.640)</option>
                             </select>
                         </div>
-                        <div class="form-group">
-                            <label>SDR Device</label>
-                            <select id="aprsDevice">
-                                <option value="">Loading devices...</option>
-                            </select>
-                        </div>
                         <div class="form-group">
                             <label>Gain (dB)</label>
                             <input type="text" id="aprsGain" value="40" placeholder="40">