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
Doorbird auth and event data #16504
Doorbird auth and event data #16504
Conversation
Add video URLs as event data to make them available to event handlers and automations.
homeassistant/components/doorbird.py
Outdated
@@ -126,16 +142,20 @@ def subscribe_events(hass, doorstation): | |||
|
|||
doorstation.device.subscribe_notification(event_type, url) | |||
|
|||
#Register a dummy listener so event is listed in GUI |
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.
block comment should start with '# '
homeassistant/components/doorbird.py
Outdated
import voluptuous as vol | ||
|
||
from homeassistant.components.http import HomeAssistantView | ||
from homeassistant.components.http import KEY_AUTHENTICATED, HomeAssistantView |
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.
'homeassistant.components.http.KEY_AUTHENTICATED' imported but unused
homeassistant/components/doorbird.py
Outdated
|
||
import asyncio |
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.
'asyncio' imported but unused
Description:
Require token to be used as auth by Doorbird devices . Add video URLs as event data to make them available to event handlers and automations.
Related issue (if applicable): #15376
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.io#6245
Example entry for
configuration.yaml
(if applicable):Checklist:
tox
. Your PR cannot be merged unless tests passIf user exposed functionality or configuration variables are added/changed:
If the code communicates with devices, web services, or third-party tools:
REQUIREMENTS
variable (example).requirements_all.txt
by runningscript/gen_requirements_all.py
..coveragerc
.If the code does not interact with devices: