Skip to content

Fix binary_sensor.template example#2877

Merged
fabaff merged 1 commit into
home-assistant:nextfrom
sdague:binary_sensor_fix
Jun 23, 2017
Merged

Fix binary_sensor.template example#2877
fabaff merged 1 commit into
home-assistant:nextfrom
sdague:binary_sensor_fix

Conversation

@sdague
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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