Merge pull request #64 from konnected-io/wait-for-wifi

Wait until WiFi is connected before starting the gdo loop.

components/secplus_gdo/secplus_gdo.cpp

@@ -133,8 +133,20 @@ namespace secplus_gdo {
 
         gdo_init(&gdo_conf);
         gdo_get_status(&this->status_);
-        gdo_start(gdo_event_handler, this);
-        ESP_LOGI(TAG, "secplus GDO started!");
+        if (this->start_gdo_) {
+            gdo_start(gdo_event_handler, this);
+            ESP_LOGI(TAG, "secplus GDO started!");
+        } else {
+            // check every 500ms for readiness before starting GDO
+            this->set_interval("gdo_start", 500, [=]() {
+                if (this->start_gdo_) {
+                    gdo_start(gdo_event_handler, this);
+                    ESP_LOGI(TAG, "secplus GDO started!");
+                    this->cancel_interval("gdo_start");
+                }
+            });
+
+        }
     }
 
     void GDOComponent::dump_config() {

components/secplus_gdo/secplus_gdo.h

@@ -33,6 +33,8 @@ namespace secplus_gdo {
         void loop() override {};
         void dump_config() override;
         void on_shutdown() override { gdo_deinit(); }
+        void start_gdo() { start_gdo_ = true; }
+
         // Use Late priority so we do not start the GDO lib until all saved preferences are loaded
         float get_setup_priority() const override { return setup_priority::LATE; }
 
@@ -103,6 +105,7 @@ namespace secplus_gdo {
         GDOSelect*                                   protocol_select_{nullptr};
         GDOSwitch*                                   learn_switch_{nullptr};
         GDOSwitch*                                   toggle_only_switch_{nullptr};
+        bool                                         start_gdo_{false};
 
     }; // GDOComponent
 } // namespace secplus_gdo

packages/secplus-gdo.yaml

@@ -152,3 +152,7 @@ number:
     type: rolling_code
     mode: box
     internal: true
+
+wifi:
+  on_connect:
+    lambda: id(gdo_blaq).start_gdo();