-
-
Notifications
You must be signed in to change notification settings - Fork 7.7k
Script - wait_template 'trigger.entity_id' support #3593
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Looks good to me!
@cdce8p Could you please rebase this one onto the |
source/_docs/scripts.markdown
Outdated
@@ -98,7 +98,7 @@ wait_template: "{{ is_state(trigger.entity_id, 'on') }}" | |||
``` | |||
{% endraw %} | |||
|
|||
It is also possible to use dummy variables, e.g. in scripts, when using `wait_template`. | |||
It is also possible to use dummy variables, e.g., in scripts, when using `wait_template`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Check out Link. Both versions are possible. I just tend to like the British one better.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is OK to like the British one better, but we have a policy on using American English only.
@frenck I would suggest to leave it at |
Description:
Added
trigger.entity_id
andvariables
support for wait_templatePull request in home-assistant: home-assistant/core#9807