Skip to content

Doorbird auth and event data#6245

Merged
arsaboo merged 1 commit intohome-assistant:nextfrom
oblogic7:doorbird_auth
Nov 1, 2018
Merged

Doorbird auth and event data#6245
arsaboo merged 1 commit intohome-assistant:nextfrom
oblogic7:doorbird_auth

Conversation

@oblogic7
Copy link
Copy Markdown
Contributor

@oblogic7 oblogic7 commented Sep 9, 2018

Adds token/auth config details as well as event_data details.

Description:
Update documentation to match Doorbird auth and event data changes.

Pull request in home-assistant (if applicable): home-assistant/core#14933

Checklist:

  • Branch: next is for changes and new documentation that will go public with the next home-assistant release. Fixes, changes and adjustments for the current release should be created against current.
  • The documentation follow the standards.

@ghost ghost added the to-do label Sep 9, 2018
@oblogic7
Copy link
Copy Markdown
Contributor Author

oblogic7 commented Sep 9, 2018

Replaces #6231

@frenck frenck added new-feature This PR adds documentation for a new Home Assistant feature to an existing integration ready-for-review This PR needs to be reviewed next This PR goes into the next branch has-parent This PR has a parent PR in another repo and removed to-do labels Sep 10, 2018
Copy link
Copy Markdown
Contributor

@DubhAd DubhAd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@DubhAd DubhAd added awaits-parent Awaits the merge of an parent PR and removed ready-for-review This PR needs to be reviewed labels Sep 29, 2018
@oblogic7
Copy link
Copy Markdown
Contributor Author

What parent PR is this waiting on?

@frenck
Copy link
Copy Markdown
Member

frenck commented Oct 21, 2018

@oblogic7 The one you've listed in the OP: home-assistant/core#16504

@oblogic7
Copy link
Copy Markdown
Contributor Author

oblogic7 commented Oct 21, 2018 via email

@frenck
Copy link
Copy Markdown
Member

frenck commented Oct 23, 2018

@oblogic7 The parent PR has been closed, but since you mentioned something about new PR, I wonder, is this documentation PR still relevant?

@oblogic7
Copy link
Copy Markdown
Contributor Author

oblogic7 commented Oct 23, 2018 via email

@wolph
Copy link
Copy Markdown
Contributor

wolph commented Nov 1, 2018

Since home-assistant/core#14933 has been merged, this needs to be merged as well :)

Ping @balloob

@arsaboo arsaboo merged commit fe26c4e into home-assistant:next Nov 1, 2018
@ghost ghost removed the awaits-parent Awaits the merge of an parent PR label Nov 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

has-parent This PR has a parent PR in another repo new-feature This PR adds documentation for a new Home Assistant feature to an existing integration next This PR goes into the next branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants