-
-
Notifications
You must be signed in to change notification settings - Fork 37.7k
Improve trigger descriptions #157643
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
Improve trigger descriptions #157643
Changes from 2 commits
b42bfc8
bbcd2b1
deb8692
bc2fb7a
d69ff82
895cb66
5014466
eb8a527
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -112,44 +112,44 @@ | |
| "title": "Assist satellite", | ||
| "triggers": { | ||
| "idle": { | ||
| "description": "Triggers when an Assist satellite becomes idle.", | ||
| "description": "Triggers when one or several voice assistant satellites became idle after having processed a command.", | ||
|
MartinHjelmare marked this conversation as resolved.
Outdated
|
||
| "fields": { | ||
| "behavior": { | ||
| "description": "[%key:component::assist_satellite::common::trigger_behavior_description%]", | ||
| "name": "[%key:component::assist_satellite::common::trigger_behavior_name%]" | ||
| } | ||
| }, | ||
| "name": "When an Assist satellite becomes idle" | ||
| "name": "Satellite became idle" | ||
|
||
| }, | ||
| "listening": { | ||
| "description": "Triggers when an Assist satellite starts listening.", | ||
| "description": "Triggers when one or several voice assistant satellites started listening for a command from someone.", | ||
|
MartinHjelmare marked this conversation as resolved.
Outdated
|
||
| "fields": { | ||
| "behavior": { | ||
| "description": "[%key:component::assist_satellite::common::trigger_behavior_description%]", | ||
| "name": "[%key:component::assist_satellite::common::trigger_behavior_name%]" | ||
| } | ||
| }, | ||
| "name": "When an Assist satellite starts listening" | ||
| "name": "Satellite started listening" | ||
|
MartinHjelmare marked this conversation as resolved.
|
||
| }, | ||
| "processing": { | ||
| "description": "Triggers when an Assist satellite is processing.", | ||
| "description": "Triggers when one or several voice assistant satellites started processing a command after having heard it.", | ||
| "fields": { | ||
| "behavior": { | ||
| "description": "[%key:component::assist_satellite::common::trigger_behavior_description%]", | ||
| "name": "[%key:component::assist_satellite::common::trigger_behavior_name%]" | ||
| } | ||
| }, | ||
| "name": "When an Assist satellite is processing" | ||
| "name": "Satellite started processing" | ||
|
||
| }, | ||
| "responding": { | ||
| "description": "Triggers when an Assist satellite is responding.", | ||
| "description": "Triggers when one or several voice assistant satellites started responding to a command after having processed it, or starts to announce something.", | ||
| "fields": { | ||
| "behavior": { | ||
| "description": "[%key:component::assist_satellite::common::trigger_behavior_description%]", | ||
| "name": "[%key:component::assist_satellite::common::trigger_behavior_name%]" | ||
| } | ||
| }, | ||
| "name": "When an Assist satellite is responding" | ||
| "name": "Satellite started responding" | ||
|
||
| } | ||
| } | ||
| } | ||
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -299,54 +299,54 @@ | |||||
| "title": "Climate", | ||||||
| "triggers": { | ||||||
| "started_cooling": { | ||||||
| "description": "Triggers when a climate started cooling.", | ||||||
| "description": "Triggers when one or several climate control devices started cooling.", | ||||||
| "fields": { | ||||||
| "behavior": { | ||||||
| "description": "[%key:component::climate::common::trigger_behavior_description%]", | ||||||
| "name": "[%key:component::climate::common::trigger_behavior_name%]" | ||||||
| } | ||||||
| }, | ||||||
| "name": "When a climate started cooling" | ||||||
| "name": "Climate control device started cooling" | ||||||
|
||||||
| }, | ||||||
| "started_drying": { | ||||||
| "description": "Triggers when a climate started drying.", | ||||||
| "description": "Triggers when one or several climate control devices started drying.", | ||||||
| "fields": { | ||||||
| "behavior": { | ||||||
| "description": "[%key:component::climate::common::trigger_behavior_description%]", | ||||||
| "name": "[%key:component::climate::common::trigger_behavior_name%]" | ||||||
| } | ||||||
| }, | ||||||
| "name": "When a climate started drying" | ||||||
| "name": "Climate control device started drying" | ||||||
|
||||||
| }, | ||||||
| "started_heating": { | ||||||
| "description": "Triggers when a climate starts to heat.", | ||||||
| "description": "Triggers when one or several climate control devices started heating.", | ||||||
| "fields": { | ||||||
| "behavior": { | ||||||
| "description": "[%key:component::climate::common::trigger_behavior_description%]", | ||||||
| "name": "[%key:component::climate::common::trigger_behavior_name%]" | ||||||
| } | ||||||
| }, | ||||||
| "name": "When a climate starts to heat" | ||||||
| "name": "Climate control device started heating" | ||||||
|
||||||
| }, | ||||||
| "turned_off": { | ||||||
| "description": "Triggers when a climate is turned off.", | ||||||
| "description": "Triggers when one or several climate control devices turned off.", | ||||||
| "fields": { | ||||||
| "behavior": { | ||||||
| "description": "[%key:component::climate::common::trigger_behavior_description%]", | ||||||
| "name": "[%key:component::climate::common::trigger_behavior_name%]" | ||||||
| } | ||||||
| }, | ||||||
| "name": "When a climate is turned off" | ||||||
| "name": "Climate control device turned off" | ||||||
|
||||||
| }, | ||||||
| "turned_on": { | ||||||
| "description": "Triggers when a climate is turned on.", | ||||||
| "description": "Triggers when one or several climate control devices turned on, regardless of the mode.", | ||||||
| "fields": { | ||||||
| "behavior": { | ||||||
| "description": "[%key:component::climate::common::trigger_behavior_description%]", | ||||||
| "name": "[%key:component::climate::common::trigger_behavior_name%]" | ||||||
| } | ||||||
| }, | ||||||
| "name": "When a climate is turned on" | ||||||
| "name": "Climate control device turned on" | ||||||
|
||||||
| "name": "Climate control device turned on" | |
| "name": "Climate control device turn on" |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -166,24 +166,24 @@ | |
| "title": "Fan", | ||
| "triggers": { | ||
| "turned_off": { | ||
| "description": "Triggers when a fan is turned off.", | ||
| "description": "Triggers when one or several fans turned off.", | ||
| "fields": { | ||
| "behavior": { | ||
| "description": "[%key:component::fan::common::trigger_behavior_description%]", | ||
| "name": "[%key:component::fan::common::trigger_behavior_name%]" | ||
| } | ||
| }, | ||
| "name": "When a fan is turned off" | ||
| "name": "Fan turned off" | ||
|
||
| }, | ||
| "turned_on": { | ||
| "description": "Triggers when a fan is turned on.", | ||
| "description": "Triggers when one or several fans turned on.", | ||
|
MartinHjelmare marked this conversation as resolved.
Outdated
|
||
| "fields": { | ||
| "behavior": { | ||
| "description": "[%key:component::fan::common::trigger_behavior_description%]", | ||
| "name": "[%key:component::fan::common::trigger_behavior_name%]" | ||
| } | ||
| }, | ||
| "name": "When a fan is turned on" | ||
| "name": "Fan turned on" | ||
|
||
| } | ||
| } | ||
| } | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -41,44 +41,44 @@ | |
| "title": "Lawn mower", | ||
| "triggers": { | ||
| "docked": { | ||
|
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. should we change this to "returned_to_dock", or is "docked" good enough?
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Docked is good. |
||
| "description": "Triggers when a lawn mower has docked.", | ||
| "description": "Triggers when one or several lawn mowers returned to dock.", | ||
| "fields": { | ||
| "behavior": { | ||
| "description": "[%key:component::lawn_mower::common::trigger_behavior_description%]", | ||
| "name": "[%key:component::lawn_mower::common::trigger_behavior_name%]" | ||
| } | ||
| }, | ||
| "name": "When a lawn mower has docked" | ||
| "name": "Lawn mower returned to dock" | ||
|
MartinHjelmare marked this conversation as resolved.
|
||
| }, | ||
| "errored": { | ||
|
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. should we change this to "encounted_error", or is "errored" good enough? |
||
| "description": "Triggers when a lawn mower has errored.", | ||
| "description": "Triggers when one or several lawn mowers encountered an error.", | ||
| "fields": { | ||
| "behavior": { | ||
| "description": "[%key:component::lawn_mower::common::trigger_behavior_description%]", | ||
| "name": "[%key:component::lawn_mower::common::trigger_behavior_name%]" | ||
| } | ||
| }, | ||
| "name": "When a lawn mower has errored" | ||
| "name": "Lawn mower encountered an error" | ||
|
||
| }, | ||
| "paused_mowing": { | ||
| "description": "Triggers when a lawn mower has paused mowing.", | ||
| "description": "Triggers when one or several lawn mowers paused mowing.", | ||
| "fields": { | ||
| "behavior": { | ||
| "description": "[%key:component::lawn_mower::common::trigger_behavior_description%]", | ||
| "name": "[%key:component::lawn_mower::common::trigger_behavior_name%]" | ||
| } | ||
| }, | ||
| "name": "When a lawn mower has paused mowing" | ||
| "name": "Lawn mower paused mowing" | ||
|
MartinHjelmare marked this conversation as resolved.
|
||
| }, | ||
| "started_mowing": { | ||
| "description": "Triggers when a lawn mower has started mowing.", | ||
| "description": "Triggers when one or several lawn mowers started mowing.", | ||
| "fields": { | ||
| "behavior": { | ||
| "description": "[%key:component::lawn_mower::common::trigger_behavior_description%]", | ||
| "name": "[%key:component::lawn_mower::common::trigger_behavior_name%]" | ||
| } | ||
| }, | ||
| "name": "When a lawn mower has started mowing" | ||
| "name": "Lawn mower started mowing" | ||
|
||
| } | ||
| } | ||
| } | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -510,24 +510,24 @@ | |
| "title": "Light", | ||
| "triggers": { | ||
| "turned_off": { | ||
| "description": "Triggers when a light is turned off.", | ||
| "description": "Triggers when one or several lights turned off.", | ||
| "fields": { | ||
| "behavior": { | ||
| "description": "[%key:component::light::common::trigger_behavior_description%]", | ||
| "name": "[%key:component::light::common::trigger_behavior_name%]" | ||
| } | ||
| }, | ||
| "name": "When a light is turned off" | ||
| "name": "Light turned off" | ||
|
||
| }, | ||
| "turned_on": { | ||
| "description": "Triggers when a light is turned on.", | ||
| "description": "Triggers when one or several lights turned on.", | ||
| "fields": { | ||
| "behavior": { | ||
| "description": "[%key:component::light::common::trigger_behavior_description%]", | ||
| "name": "[%key:component::light::common::trigger_behavior_name%]" | ||
| } | ||
| }, | ||
| "name": "When a light is turned on" | ||
| "name": "Light turned on" | ||
|
||
| } | ||
| } | ||
| } | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -381,14 +381,14 @@ | |
| "title": "Media player", | ||
| "triggers": { | ||
| "stopped_playing": { | ||
| "description": "Triggers when a media player stops playing.", | ||
| "description": "Triggers when one or several media players stopped playing media.", | ||
| "fields": { | ||
| "behavior": { | ||
| "description": "[%key:component::media_player::common::trigger_behavior_description%]", | ||
| "name": "[%key:component::media_player::common::trigger_behavior_name%]" | ||
| } | ||
| }, | ||
| "name": "When a media player stops playing" | ||
| "name": "Media player stopped playing" | ||
|
||
| } | ||
| } | ||
| } | ||
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.
The keys for assist_satellite are not consistent with other domains