fix postDataToAirGradient set false after device power up and con…

…nected to WiFi
airgradienthq · Apr 22, 2024 · 2d96fc2 · 2d96fc2
1 parent 7561017
commit 2d96fc2
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 3 deletions.
diff --git a/src/AgWiFiConnector.cpp b/src/AgWiFiConnector.cpp
@@ -46,6 +46,7 @@ bool WifiConnector::connect(void) {
   }
 
   WIFI()->setConfigPortalBlocking(false);
+  WIFI()->setConnectTimeout(15);
   WIFI()->setTimeout(WIFI_CONNECT_COUNTDOWN_MAX);
 
 #ifdef ESP32
@@ -148,9 +149,15 @@ bool WifiConnector::connect(void) {
     hasConfig = true;
     logInfo("WiFi Connected: " + WiFi.SSID() + " IP: " + localIpStr());
 
-    String result = String(postToAg.getValue());
-    logInfo("Post to AirGradient Configure: " + result);
-    config.setPostToAirGradient(result != "T");
+    if (hasPortalConfig) {
+      String result = String(postToAg.getValue());
+      logInfo("Setting postToAirGradient set from " +
+              String(config.isPostDataToAirGradient() ? "True" : "False") +
+              String(" to ") + String(result != "T" ? "True" : "False") +
+              String(" successful"));
+      config.setPostToAirGradient(result != "T");
+    }
+    hasPortalConfig = false;
   }
 #else
   _wifiProcess();
@@ -226,6 +233,7 @@ void WifiConnector::_wifiSaveParamCallback(void) {
   sm.ledAnimationInit();
   sm.handleLeds(AgStateMachineWiFiManagerStaConnecting);
   sm.setDisplayState(AgStateMachineWiFiManagerStaConnecting);
+  hasPortalConfig = true;
 }
 
 /**

diff --git a/src/AgWiFiConnector.h b/src/AgWiFiConnector.h
@@ -23,6 +23,7 @@ class WifiConnector : public PrintLog {
   void *wifi = NULL;
   bool hasConfig;
   uint32_t lastRetry;
+  bool hasPortalConfig = false;
 
   bool wifiClientConnected(void);