-
-
Notifications
You must be signed in to change notification settings - Fork 36.3k
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 6 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": { | ||
|
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. The keys for assist_satellite are not consistent with other domains |
||
| "description": "Triggers when an Assist satellite becomes idle.", | ||
| "description": "Triggers after one or more voice assistant satellites become idle after having processed a command.", | ||
| "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 after one or more voice assistant satellites start listening for a command from someone.", | ||
| "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.
Show resolved
Hide resolved
|
||
| }, | ||
| "processing": { | ||
| "description": "Triggers when an Assist satellite is processing.", | ||
| "description": "Triggers after one or more voice assistant satellites start 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 after one or more voice assistant satellites start responding to a command after having processed it, or start to announce something.", | ||
emontnemery marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
| "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 |
|---|---|---|
|
|
@@ -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 after one or more lawn mowers return 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.
Show resolved
Hide 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 after one or more lawn mowers encounter 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 after one or more lawn mowers pause 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.
Show resolved
Hide resolved
|
||
| }, | ||
| "started_mowing": { | ||
| "description": "Triggers when a lawn mower has started mowing.", | ||
| "description": "Triggers after one or more lawn mowers start 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" | ||
| } | ||
| } | ||
| } | ||
Uh oh!
There was an error while loading. Please reload this page.