diff --git a/Co2Huzzah/Co2Huzzah.ino b/Co2Huzzah/Co2Huzzah.ino index 75870e9b3b6cbd29add7cd3d0ecf15cc47126a75..c177fddae8094ed7a13619e399d6a45093f09922 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 0000000000000000000000000000000000000000..232de120d1235713ce70898fce6234a9d6449852 --- /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 + +