-
-
Notifications
You must be signed in to change notification settings - Fork 32.1k
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
Device tracker for o2box #890
Conversation
self.password = config[CONF_PASSWORD] | ||
|
||
# Establish a connection to the FRITZ!Box | ||
self.o2_box = O2Box(host=self.host, routerpassword=self.password) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What happens here if host and password are not defined in the configuration?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
Addressed the comments, updated to dev |
Can you add your file to |
This is an automated response to help you succeed in getting this PR merged.
If your code communicates with devices:
If your code does not depend on external devices:
|
Closing this PR as it has gone stale. |
This PR adds support for the O2Box, similar to the one shown here:
https://hilfe.o2online.de/docs/DOC-1332