Update documentation for fleet-based communication implementation

Author: dorkmo

TankAlarm-112025-Client-BluesOpta/FLEET_IMPLEMENTATION_SUMMARY.md

@@ -152,10 +152,6 @@ The Tank Alarm system has been updated to use **Blues Notehub fleet-based device
   "sampleSeconds": 300,
   "reportHour": 5,
   "reportMinute": 0,
-  "sms": {
-    "primary": "+12223334444",
-    "secondary": "+15556667777"
-  },
   "dailyEmail": "[email protected]",
   "tanks": [
     {
@@ -174,6 +170,8 @@ The Tank Alarm system has been updated to use **Blues Notehub fleet-based device
 }
 ```
 
+SMS recipients now reside exclusively in the server configuration. Client-side `alarmSms` flags simply signal whether a tank should request SMS escalation when it triggers an alarm.
+
 ### Server Config Example
 ```json
 {

TankAlarm-112025-Client-BluesOpta/MIGRATION_GUIDE.md

@@ -147,7 +147,7 @@ For each client:
 
 4. **Review other settings:**
    - Verify all tank configurations
-   - Confirm SMS and email settings
+   - Confirm email recipients (SMS numbers now live on the server)
    - Check sample intervals
 
 5. **Click "Send Config to Client"**

TankAlarm-112025-Client-BluesOpta/TankAlarm-112025-Client-BluesOpta.ino

@@ -161,8 +161,6 @@ struct ClientConfig {
   char siteName[32];
   char deviceLabel[24];
   char serverFleet[32]; // Target fleet name for server (e.g., "tankalarm-server")
-  char smsPrimary[20];
-  char smsSecondary[20];
   char dailyEmail[64];
   uint16_t sampleSeconds;
   uint8_t reportHour;
@@ -369,8 +367,6 @@ static void createDefaultConfig(ClientConfig &cfg) {
   strlcpy(cfg.siteName, "Opta Tank Site", sizeof(cfg.siteName));
   strlcpy(cfg.deviceLabel, "Client-112025", sizeof(cfg.deviceLabel));
   strlcpy(cfg.serverFleet, "tankalarm-server", sizeof(cfg.serverFleet));
-  strlcpy(cfg.smsPrimary, "+12223334444", sizeof(cfg.smsPrimary));
-  strlcpy(cfg.smsSecondary, "+15556667777", sizeof(cfg.smsSecondary));
   strlcpy(cfg.dailyEmail, "[email protected]", sizeof(cfg.dailyEmail));
   cfg.sampleSeconds = DEFAULT_SAMPLE_SECONDS;
   cfg.reportHour = DEFAULT_REPORT_HOUR;
@@ -417,8 +413,6 @@ static bool loadConfigFromFlash(ClientConfig &cfg) {
   strlcpy(cfg.siteName, doc["site"].as<const char *>() ? doc["site"].as<const char *>() : "", sizeof(cfg.siteName));
   strlcpy(cfg.deviceLabel, doc["deviceLabel"].as<const char *>() ? doc["deviceLabel"].as<const char *>() : "", sizeof(cfg.deviceLabel));
   strlcpy(cfg.serverFleet, doc["serverFleet"].as<const char *>() ? doc["serverFleet"].as<const char *>() : "", sizeof(cfg.serverFleet));
-  strlcpy(cfg.smsPrimary, doc["sms"]["primary"].as<const char *>() ? doc["sms"]["primary"].as<const char *>() : "", sizeof(cfg.smsPrimary));
-  strlcpy(cfg.smsSecondary, doc["sms"]["secondary"].as<const char *>() ? doc["sms"]["secondary"].as<const char *>() : "", sizeof(cfg.smsSecondary));
   strlcpy(cfg.dailyEmail, doc["dailyEmail"].as<const char *>() ? doc["dailyEmail"].as<const char *>() : "", sizeof(cfg.dailyEmail));
 
   cfg.sampleSeconds = doc["sampleSeconds"].is<uint16_t>() ? doc["sampleSeconds"].as<uint16_t>() : DEFAULT_SAMPLE_SECONDS;
@@ -465,10 +459,6 @@ static bool saveConfigToFlash(const ClientConfig &cfg) {
   doc["reportMinute"] = cfg.reportMinute;
   doc["dailyEmail"] = cfg.dailyEmail;
 
-  JsonObject smsObj = doc.createNestedObject("sms");
-  smsObj["primary"] = cfg.smsPrimary;
-  smsObj["secondary"] = cfg.smsSecondary;
-
   JsonArray tanks = doc.createNestedArray("tanks");
   for (uint8_t i = 0; i < cfg.tankCount; ++i) {
     JsonObject t = tanks.createNestedObject();
@@ -775,14 +765,6 @@ static void applyConfigUpdate(const JsonDocument &doc) {
   if (doc.containsKey("reportMinute")) {
     gConfig.reportMinute = doc["reportMinute"].as<uint8_t>();
   }
-  if (doc.containsKey("sms")) {
-    if (doc["sms"].containsKey("primary")) {
-      strlcpy(gConfig.smsPrimary, doc["sms"]["primary"].as<const char *>(), sizeof(gConfig.smsPrimary));
-    }
-    if (doc["sms"].containsKey("secondary")) {
-      strlcpy(gConfig.smsSecondary, doc["sms"]["secondary"].as<const char *>(), sizeof(gConfig.smsSecondary));
-    }
-  }
   if (doc.containsKey("dailyEmail")) {
     strlcpy(gConfig.dailyEmail, doc["dailyEmail"].as<const char *>(), sizeof(gConfig.dailyEmail));
   }
@@ -1256,8 +1238,6 @@ static void sendAlarm(uint8_t idx, const char *alarmType, float inches) {
     doc["highThreshold"] = cfg.highAlarmInches;
     doc["lowThreshold"] = cfg.lowAlarmInches;
     doc["smsEnabled"] = allowSmsEscalation;
-    doc["smsPrimary"] = gConfig.smsPrimary;
-    doc["smsSecondary"] = gConfig.smsSecondary;
     doc["time"] = currentEpoch();
 
     publishNote(ALARM_FILE, doc, true);

TankAlarm-112025-Server-BluesOpta/FLEET_SETUP.md

@@ -49,10 +49,6 @@ The client configuration file (`/client_config.json` on LittleFS) includes:
   "sampleSeconds": 300,
   "reportHour": 5,
   "reportMinute": 0,
-  "sms": {
-    "primary": "+12223334444",
-    "secondary": "+15556667777"
-  },
   "dailyEmail": "[email protected]",
   "tanks": [
     {
@@ -74,6 +70,8 @@ The client configuration file (`/client_config.json` on LittleFS) includes:
 }
 ```
 
+Server-managed SMS contacts have been removed from the client schema; per-tank `alarmSms` flags now simply request that the server escalate alerts via its own contact list.
+
 **Key field:** `serverFleet` - Must match the server's fleet name in Notehub
 
 ### Server Configuration

TankAlarm-112025-Server-BluesOpta/TankAlarm-112025-Server-BluesOpta.ino

@@ -266,8 +266,6 @@ static const char CONFIG_GENERATOR_HTML[] PROGMEM = R"HTML(
           <label class="field"><span>Sample Seconds</span><input id="sampleSeconds" type="number" value="300"></label>
           <label class="field"><span>Report Hour</span><input id="reportHour" type="number" value="5"></label>
           <label class="field"><span>Report Minute</span><input id="reportMinute" type="number" value="0"></label>
-          <label class="field"><span>SMS Primary</span><input id="smsPrimary" type="text"></label>
-          <label class="field"><span>SMS Secondary</span><input id="smsSecondary" type="text"></label>
           <label class="field"><span>Daily Email</span><input id="dailyEmail" type="email"></label>
         </div>
         
@@ -420,10 +418,6 @@ static const char CONFIG_GENERATOR_HTML[] PROGMEM = R"HTML(
         sampleSeconds: parseInt(document.getElementById('sampleSeconds').value, 10) || 300,
         reportHour: parseInt(document.getElementById('reportHour').value, 10) || 5,
         reportMinute: parseInt(document.getElementById('reportMinute').value, 10) || 0,
-        sms: {
-          primary: document.getElementById('smsPrimary').value.trim(),
-          secondary: document.getElementById('smsSecondary').value.trim()
-        },
         dailyEmail: document.getElementById('dailyEmail').value.trim(),
         tanks: []
       };
@@ -1078,6 +1072,8 @@ static const char DASHBOARD_HTML[] PROGMEM = R"HTML(
       }
 
       function syncServerSettings(serverInfo) {
+        els.smsPrimary.value = valueOr(serverInfo && serverInfo.smsPrimary, '');
+        els.smsSecondary.value = valueOr(serverInfo && serverInfo.smsSecondary, '');
         els.smsHighToggle.checked = !!valueOr(serverInfo && serverInfo.smsOnHigh, true);
         els.smsLowToggle.checked = !!valueOr(serverInfo && serverInfo.smsOnLow, true);
         els.smsClearToggle.checked = !!valueOr(serverInfo && serverInfo.smsOnClear, false);
@@ -1197,10 +1193,6 @@ static const char DASHBOARD_HTML[] PROGMEM = R"HTML(
           sampleSeconds: 300,
           reportHour: 5,
           reportMinute: 0,
-          sms: {
-            primary: serverDefaults.smsPrimary || '',
-            secondary: serverDefaults.smsSecondary || ''
-          },
           dailyEmail: serverDefaults.dailyEmail || '',
           tanks: tankList.length ? tankList.map(t => ({
             id: t.tank || 'A',
@@ -1284,9 +1276,6 @@ static const char DASHBOARD_HTML[] PROGMEM = R"HTML(
     els.sampleSeconds.value = valueOr(config.sampleSeconds, 300);
     els.reportHour.value = valueOr(config.reportHour, 5);
     els.reportMinute.value = valueOr(config.reportMinute, 0);
-    const smsConfig = config.sms || {};
-    els.smsPrimary.value = smsConfig.primary || '';
-    els.smsSecondary.value = smsConfig.secondary || '';
         els.dailyEmail.value = config.dailyEmail || '';
         populateTankRows(config.tanks);
 
@@ -1300,11 +1289,6 @@ static const char DASHBOARD_HTML[] PROGMEM = R"HTML(
       }
 
       function collectConfig() {
-        const sms = {
-          primary: els.smsPrimary.value.trim(),
-          secondary: els.smsSecondary.value.trim()
-        };
-
         const tanks = [];
         els.tankBody.querySelectorAll('tr').forEach(row => {
           const tank = {
@@ -1336,7 +1320,6 @@ static const char DASHBOARD_HTML[] PROGMEM = R"HTML(
           sampleSeconds: parseInt(els.sampleSeconds.value, 10) || 300,
           reportHour: parseInt(els.reportHour.value, 10) || 5,
           reportMinute: parseInt(els.reportMinute.value, 10) || 0,
-          sms,
           dailyEmail: els.dailyEmail.value.trim(),
           tanks
         };
@@ -1346,6 +1329,8 @@ static const char DASHBOARD_HTML[] PROGMEM = R"HTML(
 
       function collectServerSettings() {
         return {
+          smsPrimary: els.smsPrimary.value.trim(),
+          smsSecondary: els.smsSecondary.value.trim(),
           smsOnHigh: !!els.smsHighToggle.checked,
           smsOnLow: !!els.smsLowToggle.checked,
           smsOnClear: !!els.smsClearToggle.checked