search

skrollme / homebridge-eveatmo

Homebridge plugin which adds a Netatmo weatherstation as HomeKit device and tries to act like Elgato Eve Room/Weather
Apache License 2.0
69 stars 7 forks source link

Reduce log output #63

Closed smhex closed 2 years ago

smhex commented 2 years ago

Is your feature request related to a problem? Please describe. For every data update of my weather station the following log output is shown:

[Netatmo] Loading new data from API for: weatherstation
[Netatmo] Loading new data from API for: airquality

As I have many other plugins running the log gets filled with those messages and it is more difficult to find relevant log entries in case of errors.

Describe the solution you'd like Introduce a config switch to disable the log output for log level info or completely suppress them.

skrollme commented 2 years ago

Noted, thanks for your input

skrollme commented 2 years ago

Thank you for the PR @smhex, I really appreciate it 👌 In my limited time I focus on refactoring the plugin to use a new auth-mechanism (see #62 ). But when this is done, I will take a look at your PR

smhex commented 2 years ago

Don't worry! I can always create a patch for my own installation. If you need help with testing the new login method just tell.

skrollme commented 2 years ago

Since the authentication-topic (https://github.com/skrollme/homebridge-eveatmo/issues/62) is hopefully done now, I will soon take a look your PR, @smhex

skrollme commented 2 years ago

I just merged your PR and released it as 1.0.1. Thanks for your contribution @smhex 👌

smhex commented 2 years ago

@skrollme thank you for this nice plugin!