From 4353c9ebf9a4399a9b1034f0595559fc93756f83 Mon Sep 17 00:00:00 2001
From: Ian Renton <ian@ianrenton.com>
Date: Sat, 25 Jul 2026 14:43:16 +0100
Subject: [PATCH] Intermediate reload of map data if QRZ/HamQTH credentials are
 supplied

---
 templates/add_spot.html   |  2 +-
 templates/alerts.html     |  2 +-
 templates/bands.html      |  4 ++--
 templates/base.html       | 10 +++++-----
 templates/conditions.html |  2 +-
 templates/map.html        |  4 ++--
 templates/spots.html      |  4 ++--
 templates/status.html     |  2 +-
 webassets/js/map.js       | 24 ++++++++++++++++++++++--
 9 files changed, 37 insertions(+), 17 deletions(-)

diff --git a/templates/add_spot.html b/templates/add_spot.html
index c150255..863335b 100644
--- a/templates/add_spot.html
+++ b/templates/add_spot.html
@@ -76,7 +76,7 @@
 
 </div>
 
-<script src="/js/add-spot.js?v=1784983394"></script>
+<script src="/js/add-spot.js?v=1784986996"></script>
 <script>$(document).ready(function () {
     $("#nav-link-add-spot").addClass("active");
 }); <!-- highlight active page in nav --></script>
diff --git a/templates/alerts.html b/templates/alerts.html
index 1eb66d2..9d20346 100644
--- a/templates/alerts.html
+++ b/templates/alerts.html
@@ -75,7 +75,7 @@
 
 </div>
 
-<script src="/js/alerts.js?v=1784983394"></script>
+<script src="/js/alerts.js?v=1784986996"></script>
 <script>$(document).ready(function () {
     $("#nav-link-alerts").addClass("active");
 }); <!-- highlight active page in nav --></script>
diff --git a/templates/bands.html b/templates/bands.html
index 15fa39a..f2a2083 100644
--- a/templates/bands.html
+++ b/templates/bands.html
@@ -75,8 +75,8 @@
 <script>
     let spotProvidersEnabledByDefault = {% raw json_encode(web_ui_options["spot-providers-enabled-by-default"]) %};
 </script>
-<script src="/js/spotsbandsandmap.js?v=1784983394"></script>
-<script src="/js/bands.js?v=1784983394"></script>
+<script src="/js/spotsbandsandmap.js?v=1784986996"></script>
+<script src="/js/bands.js?v=1784986996"></script>
 <script>$(document).ready(function () {
     $("#nav-link-bands").addClass("active");
 }); <!-- highlight active page in nav --></script>
diff --git a/templates/base.html b/templates/base.html
index eafddda..0294bcc 100644
--- a/templates/base.html
+++ b/templates/base.html
@@ -1,6 +1,6 @@
 {% extends "skeleton.html" %}
 {% block head_extra %}
-<link rel="stylesheet" href="/css/style.css?v=1784983394" type="text/css">
+<link rel="stylesheet" href="/css/style.css?v=1784986996" type="text/css">
 <link href="/vendor/css/bootstrap-5.3.8.min.css" rel="stylesheet">
 <link href="/vendor/css/fontawesome-6.7.2.min.css" rel="stylesheet">
 <link href="/vendor/css/solid-6.7.2.min.css" rel="stylesheet">
@@ -10,10 +10,10 @@
 <script src="/vendor/js/bootstrap-5.3.8.bundle.min.js"></script>
 <script src="/vendor/js/tinycolor2-1.6.0.min.js"></script>
 
-<script src="/js/utils.js?v=1784983394"></script>
-<script src="/js/ui-ham.js?v=1784983394"></script>
-<script src="/js/geo.js?v=1784983394"></script>
-<script src="/js/common.js?v=1784983394"></script>
+<script src="/js/utils.js?v=1784986996"></script>
+<script src="/js/ui-ham.js?v=1784986996"></script>
+<script src="/js/geo.js?v=1784986996"></script>
+<script src="/js/common.js?v=1784986996"></script>
 {% end %}
 {% block body %}
 <div class="container">
diff --git a/templates/conditions.html b/templates/conditions.html
index ae6075a..a52a48a 100644
--- a/templates/conditions.html
+++ b/templates/conditions.html
@@ -284,7 +284,7 @@
 </div>
 
 <script src="/vendor/js/chart-4.4.9.umd.min.js"></script>
-<script src="/js/conditions.js?v=1784983394"></script>
+<script src="/js/conditions.js?v=1784986996"></script>
 <script>$(document).ready(function () {
     $("#nav-link-conditions").addClass("active");
 }); <!-- highlight active page in nav --></script>
diff --git a/templates/map.html b/templates/map.html
index 1fd37f1..0d4e6ee 100644
--- a/templates/map.html
+++ b/templates/map.html
@@ -95,8 +95,8 @@
 <script>
     let spotProvidersEnabledByDefault = {% raw json_encode(web_ui_options["spot-providers-enabled-by-default"]) %};
 </script>
-<script src="/js/spotsbandsandmap.js?v=1784983393"></script>
-<script src="/js/map.js?v=1784983393"></script>
+<script src="/js/spotsbandsandmap.js?v=1784986996"></script>
+<script src="/js/map.js?v=1784986996"></script>
 <script>$(document).ready(function () {
     $("#nav-link-map").addClass("active");
 }); <!-- highlight active page in nav --></script>
diff --git a/templates/spots.html b/templates/spots.html
index 04ad53b..9ccdf8a 100644
--- a/templates/spots.html
+++ b/templates/spots.html
@@ -116,8 +116,8 @@
 <script>
     let spotProvidersEnabledByDefault = {% raw json_encode(web_ui_options["spot-providers-enabled-by-default"]) %};
 </script>
-<script src="/js/spotsbandsandmap.js?v=1784983393"></script>
-<script src="/js/spots.js?v=1784983393"></script>
+<script src="/js/spotsbandsandmap.js?v=1784986996"></script>
+<script src="/js/spots.js?v=1784986996"></script>
 <script>$(document).ready(function () {
     $("#nav-link-spots").addClass("active");
 }); <!-- highlight active page in nav --></script>
diff --git a/templates/status.html b/templates/status.html
index 24c2ad9..7c1dfe4 100644
--- a/templates/status.html
+++ b/templates/status.html
@@ -59,7 +59,7 @@
     </div>
 </div>
 
-<script src="/js/status.js?v=1784983394"></script>
+<script src="/js/status.js?v=1784986996"></script>
 <script>
     $(document).ready(function () {
         $("#nav-link-status").addClass("active");
diff --git a/webassets/js/map.js b/webassets/js/map.js
index 5eba7fa..1de1a4e 100644
--- a/webassets/js/map.js
+++ b/webassets/js/map.js
@@ -38,8 +38,13 @@ function loadSpots() {
     if (evtSource != null) {
         evtSource.close();
     }
-    // On first fetch, don't include QRZ/HamQTH credentials. This will cause some inaccurate
-    // marker positions but avoids having to wait a minute or more for all the lookups to fire.
+    // Including QRZ/HamQTH lookups to improve positions causes the load to be really slow, so on first load
+    // the user would be waiting ages without data and will think it's broken. We therefore load in several
+    // stages:
+    // 1) Load without any credentials
+    // 2) (If we have credentials) reload with them, replacing what's already there,
+    // 3) Subscribe to the SSE endpoint (with credentials if we have them) so that updates come with augmented
+    // data if they can.
     $.getJSON('/api/v1/spots' + buildQueryString(false), function (jsonData) {
         // Store data
         spots = jsonData;
@@ -48,6 +53,21 @@ function loadSpots() {
         if ($("#showTerminator")[0].checked) {
             terminator.setTime();
         }
+        // Check if we have any credentials to use
+        if (getCredentialQueryString() !== "") {
+            // OK, we have credentials and have loaded once without them so the user has a basic map. Now reload
+            // with the credentials and replace what's on the map, so we can improve the data.
+            $.getJSON('/api/v1/spots' + buildQueryString(true), function (jsonData2) {
+                spots = jsonData2;
+                updateMap();
+                // Now start the ongoing SSE connection
+                startSSEConnection();
+            });
+        } else {
+            // We had no credentials with which to augment the data anyway, so just start the SSE connection
+            // now
+            startSSEConnection();
+        }
         // Start the ongoing SSE connection
         startSSEConnection();
     });