KlausMu / esp32-fan-controller

ESP32 fan controller with temperature sensor and MQTT
272 stars 48 forks source link

Dht sensor #6

Closed ffa95 closed 2 years ago

ffa95 commented 3 years ago

it's possible to use a new type of temp sensor. The new sensor is DHT type. You choose your type in config.h (DTH 11/22/21 - AM2301)

KlausMu commented 3 years ago

Thanks for your contribution, I would like to merge it. I only would like to do minor changes first, mainly formatting, turning French to English and so on. For doing this, could you please

After this I would try to change the mentioned minor things in your pull request and then to merge it. Still have to learn how this works, didn't do this before :-)

KlausMu commented 3 years ago

Could you please only add the files "sensorDHT.cpp" and "sensorDHT.h" and not the files from the Adafruit library? You should include the Adafruit library in "platformio.ini".

KlausMu commented 3 years ago

As far as i can see, you don't handle the presence of a DHT neither in mqtt.cpp nor in tft.cpp. So I assume you don't use them. But how do you want to control the ESP32? You cannot disable both mqtt and touch.