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

Add device registry to MQTT binary sensor #6605

Merged
merged 2 commits into from
Oct 8, 2018

Conversation

OttoWinter
Copy link
Member

Description:

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

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 assigned OttoWinter Oct 8, 2018
@ghost ghost added the ready-for-review This PR needs to be reviewed label Oct 8, 2018
@OttoWinter OttoWinter added the Hacktoberfest An PR on this issue (or the PR itself) is eligible towards Hacktoberfest! label Oct 8, 2018
Copy link
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.

Connections example is confusing, showing both [ blah, blah] an [[blah, blah]]

required: false
type: list, string
connections:
description: 'A list of connections of the device to the outside world as a list of tuples `[connection_type, connection_identifier]`. For example the MAC address of a network interface: `[["mac", "02:5b:26:a8:dc:12"]]`.'
Copy link
Contributor

Choose a reason for hiding this comment

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

Should that be single, or double, brackets?

Copy link
Member Author

Choose a reason for hiding this comment

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

double brackets. It's a list of 2-tuples.

Copy link
Member Author

Choose a reason for hiding this comment

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

I clarified it a bit now by including "connections": in the example config. I'm writing it in JSON format as that's the only way this config option can be used (only over MQTT discovery).

Copy link
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 has-parent This PR has a parent PR in a other repo and removed ready-for-review This PR needs to be reviewed labels Oct 8, 2018
@DubhAd DubhAd merged commit 4a1af47 into next Oct 8, 2018
@ghost ghost removed the awaits-parent Awaits the merge of an parent PR label Oct 8, 2018
@OttoWinter OttoWinter deleted the mqtt-device-registry-binary-sensor branch October 8, 2018 16:47
@frenck frenck added new-feature This PR adds documentation for a new Home Assistant feature to an existing integration next This PR goes into the next branch labels Oct 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Hacktoberfest An PR on this issue (or the PR itself) is eligible towards Hacktoberfest! has-parent This PR has a parent PR in a other 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.

3 participants