From 4b5aa10db9a2ff1ccc17502465fec7de1924b8f3 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?St=C3=A9phan=20Bernard?= <stephan.bernard@inrae.fr>
Date: Wed, 9 Nov 2022 09:39:33 +0100
Subject: [PATCH] =?UTF-8?q?Merge=20de=20la=20branche=20MQTT,=20suppression?=
 =?UTF-8?q?=20du=20passage=20=C3=A0=20l'UTC=20au=20moindre=20probl=C3=A8me?=
 =?UTF-8?q?=20NTP.=20Il=20reste=20du=20boulot=20=C3=A0=20faire=20donc=20ap?=
 =?UTF-8?q?patition=20d'un=20TODO.?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 Co2Huzzah/Co2Huzzah.ino |  5 ++--
 Co2Huzzah/TODO          | 53 +++++++++++++++++++++++++++++++++++++++++
 2 files changed, 55 insertions(+), 3 deletions(-)
 create mode 100644 Co2Huzzah/TODO

diff --git a/Co2Huzzah/Co2Huzzah.ino b/Co2Huzzah/Co2Huzzah.ino
index 75870e9..c177fdd 100644
--- a/Co2Huzzah/Co2Huzzah.ino
+++ b/Co2Huzzah/Co2Huzzah.ino
@@ -962,9 +962,8 @@ void setup()
   setInterval(120); /* Try every 2 minutes until success */
   if (!tz.setLocation(timezone)) {
     Serial.print(MSG[WARN]);
-    Serial.print(F("Timezone setting failed -> "));
-    Serial.println(timezone);
-    tz = UTC;
+    Serial.println(F("Timezone setting failed."));
+    /* tz = UTC; */
   }
   digitalWrite(BLUE_LED, HIGH); /* HIGH value turns it off */
 
diff --git a/Co2Huzzah/TODO b/Co2Huzzah/TODO
new file mode 100644
index 0000000..232de12
--- /dev/null
+++ b/Co2Huzzah/TODO
@@ -0,0 +1,53 @@
+# TODO
+
+## Désactivation de MQTT
+
+De même qu'il est possible de désactiver la sauvegarde des mesures
+dans des fichiers csv, on doit permettre de désactiver la fonctionnalité
+d'envoi des données à un broker MQTT.
+
+Actuellement, donner une adresse bidon pour le broker reste possible, mais
+oblige le nœud à tenter d'accéder au broker pendant 10s à chaque mesure.
+
+
+## Clarifier la page de paramètres Wifi
+
+La saisie de l'IP du broker MQTT a été rajoutéa à cette page,
+et le bouton "Set" concerne l'ensemble des paramètres. En réalité,
+
+- une valeur vide pour l'IP du broker laisse le broker MQTT inchangé,
+- une valeur vide pour le ssid du point d'accès laisse les paramètres
+  du client wifi inchangés.
+  
+Néanmoins ça n'est ni documenté ni intuitif. On pourrait imaginer deux boutons
+set, comme sur la page de la date et l'heure (selon qu'on utilise l'interface
+pour régler manuellement la date et l'heure ou pour paramétrer le client NTP).
+
+
+## Optimiser le code
+
+Il y a beaucoup de choses dans ce code car ce nœud capteur fait beaucoup
+trop de choses à la fois :
+
+- Point d'accès Wifi
+- Client Wifi (en même temps)
+- Client NTP
+- Client MQTT
+- Serveur http ET https
+- NÅ“ud capteur
+- Sauvegarde des données capteur dans des fichiers
+- le café (ah non, dommage)
+- …
+
+Du coup le code est un peu fouills. Il serait bien de faire un peu de relecture
+pour le reprendre, le rendre plus clair et l'optimiser.
+
+Quelques pistes :
+
+- regrouper les *handlers* "/sensor", "/mqttbroker", "/ap", "/status", "/dtz",
+  et "/sensorsetup" en un seul *handler* (car ils n'envoient que des valeurs
+  qu'on a sous la main, contrairement à "/scan" ou "/listfiles").
+
+- permettre de désactiver en dur (#ifdef SERIAL_OUTPUT) les sorties console
+
+
-- 
GitLab