search

alcar21 / WemosEM

Cheap power monitor (STC013) with Home Assistant Discover and Wemos D1 Mini / PRO (esp8266)
113 stars 23 forks source link

Fixes for a couple of issues #8

Closed NorthernMan54 closed 4 years ago

NorthernMan54 commented 4 years ago

This pull request contains fixes and tweaks for a couple of issues. This was tested in my environment with esp8266 version 2.4.2 and version 2.6.3 using Arduino and not platform.io

Fixes

1 - Added a em_read(true); to the end of the setup function in WemosEM.ino to remove invalid readings

2 - Removed homeassistant MQTT publish ( mqtt_wifi_serial_support.h ) as it isn't needed ( Maybe put this back )

3 - Removed saveConfig(); after every read of current ( power_meter_support.h ) was worried about wearing out flash memory on esp8266

4 - Fixed serial output for MQTT publish ( power_meter_support.h )

5 - Fixed Mqtt_enabled not being restarted after reboot ( settings.h )

6 - Fix for exception during restart for a new esp8266 ( variables.h )

7 - Added save config before restart ( webserver_support.h )

8 - Fixed North American time zones ( webserver_support.h ) ie -5

9 - Removed restart after saving System ( webserver_support.h )

10 - Fix for ESP8266 2.6.3 in ( WiFiManager.cpp, parse.js, template.h and WiFiManager.h )

Sorry my editor ( Atom ) go rid of the extra space characters

PS I'm not a c++ developer, but can muddle my way thru.

alcar21 commented 4 years ago

Thank you. Good Job. Calm, I'm java developer :blush:

rantanlan commented 3 years ago

The HA autodiscovery was removed with this.. any chance to get this back? Switched to a new homeassistance instance, and this broke autodiscover for me.