From 86111196d6155b6bf4eb9d032d6488a0d5211ea6 Mon Sep 17 00:00:00 2001 From: tekktrik <89490472+tekktrik@users.noreply.github.com> Date: Thu, 30 Dec 2021 12:18:04 -0500 Subject: [PATCH 1/3] Revert status LED to off after POST --- adafruit_esp32spi/adafruit_esp32spi_wifimanager.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/adafruit_esp32spi/adafruit_esp32spi_wifimanager.py b/adafruit_esp32spi/adafruit_esp32spi_wifimanager.py index 930b33c..a02cada 100755 --- a/adafruit_esp32spi/adafruit_esp32spi_wifimanager.py +++ b/adafruit_esp32spi/adafruit_esp32spi_wifimanager.py @@ -128,7 +128,7 @@ def _get_next_ap(self): def connect_normal(self): """ - Attempt a regular style WiFi connection + Attempt a regular style WiFi connection. """ failure_count = 0 (ssid, password) = self._get_next_ap() @@ -242,6 +242,7 @@ def post(self, url, **kw): self.connect() self.pixel_status((0, 0, 100)) return_val = requests.post(url, **kw) + self.pixel_status(0) return return_val def put(self, url, **kw): From fd7c932af3f8662b3b9135683430fed99df3a122 Mon Sep 17 00:00:00 2001 From: tekktrik <89490472+tekktrik@users.noreply.github.com> Date: Thu, 30 Dec 2021 12:33:46 -0500 Subject: [PATCH 2/3] Update __init__() LED argument to describe LED behavior --- adafruit_esp32spi/adafruit_esp32spi_wifimanager.py | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/adafruit_esp32spi/adafruit_esp32spi_wifimanager.py b/adafruit_esp32spi/adafruit_esp32spi_wifimanager.py index a02cada..5e3262e 100755 --- a/adafruit_esp32spi/adafruit_esp32spi_wifimanager.py +++ b/adafruit_esp32spi/adafruit_esp32spi_wifimanager.py @@ -43,7 +43,11 @@ def __init__( :param ESP_SPIcontrol esp: The ESP object we are using :param dict secrets: The WiFi and Adafruit IO secrets dict (See examples) :param status_pixel: (Optional) The pixel device - A NeoPixel, DotStar, - or RGB LED (default=None) + or RGB LED (default=None). The status LED, if given, turns red when + attempting to connect to a Wi-Fi network or create an access point, + turning green upon success. Additionally, if given, it will turn blue + when attempting an HTTP method or returning IP address, turning off + upon success. :type status_pixel: NeoPixel, DotStar, or RGB LED :param int attempts: (Optional) Failed attempts before resetting the ESP32 (default=2) :param const connection_type: (Optional) Type of WiFi connection: NORMAL or ENTERPRISE From 436700ae46a67c7ed62f86dfab735cb129df5d5c Mon Sep 17 00:00:00 2001 From: Tekktrik Date: Thu, 30 Dec 2021 09:44:31 -0800 Subject: [PATCH 3/3] Reformatted per pre-commit --- adafruit_esp32spi/adafruit_esp32spi_wifimanager.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/adafruit_esp32spi/adafruit_esp32spi_wifimanager.py b/adafruit_esp32spi/adafruit_esp32spi_wifimanager.py index 5e3262e..739f7a9 100755 --- a/adafruit_esp32spi/adafruit_esp32spi_wifimanager.py +++ b/adafruit_esp32spi/adafruit_esp32spi_wifimanager.py @@ -132,7 +132,7 @@ def _get_next_ap(self): def connect_normal(self): """ - Attempt a regular style WiFi connection. + Attempt a regular style WiFi connection. """ failure_count = 0 (ssid, password) = self._get_next_ap()