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

device_tracker.mqtt configuration.yaml is wrong #3583

Merged
merged 1 commit into from
Oct 11, 2017

Conversation

tsvi
Copy link
Contributor

@tsvi tsvi commented Oct 10, 2017

The topic to be looked up must be in quotes for the example to work.

Description:
The current documentation's example does not wrong. By putting the topic in quotes the mqtt device tracker actually does work.

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

The topic to be looked up must be in quotes for the example to work.
@fabaff fabaff added Fix/Formatting/Language/Spelling Hacktoberfest An PR on this issue (or the PR itself) is eligible towards Hacktoberfest! labels Oct 11, 2017
Copy link
Member

@fabaff fabaff left a comment

Choose a reason for hiding this comment

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

It looks more like that the tracker is not working because the topics are starting with /. I see no difference if the topic is wrapped in quotes or not. But to be on the safe side let's wrap it.

Thanks 🐦

@fabaff fabaff merged commit 9c25564 into home-assistant:current Oct 11, 2017
fabaff pushed a commit that referenced this pull request Oct 21, 2017
The topic to be looked up must be in quotes for the example to work.
@tsvi tsvi deleted the patch-1 branch October 4, 2018 10:45
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!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants