Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reduce log output #63

Closed
smhex opened this issue Sep 4, 2022 · 6 comments
Closed

Reduce log output #63

smhex opened this issue Sep 4, 2022 · 6 comments
Assignees

Comments

@smhex
Copy link
Contributor

smhex commented Sep 4, 2022

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
Copy link
Owner

skrollme commented Sep 5, 2022

Noted, thanks for your input

@skrollme
Copy link
Owner

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
Copy link
Contributor Author

smhex commented Sep 18, 2022

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
Copy link
Owner

Since the authentication-topic (#62) is hopefully done now, I will soon take a look your PR, @smhex

@skrollme
Copy link
Owner

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

@smhex
Copy link
Contributor Author

smhex commented Nov 1, 2022

@skrollme thank you for this nice plugin!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants