Skip to content

Fix binary_sensor.template example #2877

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

Merged
merged 1 commit into from
Jun 23, 2017

Conversation

sdague
Copy link
Contributor

@sdague sdague commented Jun 23, 2017

The binary_sensor.template was return states of "On" and "Off",
however that doesn't seem to work in 0.47. You need True / False,
which makes sense, given that's what the single line evals are going
to return. Update the existing example, and add another one about
computing composite occupancy from device tracker and motion sensors.

The binary_sensor.template was return states of "On" and "Off",
however that doesn't seem to work in 0.47. You need True / False,
which makes sense, given that's what the single line evals are going
to return. Update the existing example, and add another one about
computing composite occupancy from device tracker and motion sensors.
@sdague
Copy link
Contributor Author

sdague commented Jun 23, 2017

Appologies for some extra white space fixing, my editor does a whitespace clean on save so the random characters get purged. It usually makes patches better, but on docs where whitespace at end of lines is sometimes forgotten, it gets dorky.

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.

Thanks 🐦

Feel free to create PR for existing docs against current.

@fabaff fabaff merged commit ce30b5b into home-assistant:next Jun 23, 2017
@balloob
Copy link
Member

balloob commented Jun 24, 2017

Jinja2 templates need to be wrapped in {% raw %} tags or the jekyll build will fail (that's why CI failed). Fixed it in 2189bbd 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants