Remote control updates #3997
Remote control updates #3997
Conversation
|
I know it feels bad to remove a big chunk of work but in this context I find it confusing. I believe the platform documentation should explain the platform and not teach how to combine things, we have other resources for that. I considered moving it to the Cookbook but there are already similar examples, for example here. |
|
|
||
| | Service data attribute | Optional | Description | | ||
| | ---------------------- | -------- | ----------- | | ||
| | `entity_id` | yes | Only act on specific remote. Else targets all. |
There was a problem hiding this comment.
Missing determiner: on a specific remote
Use comma: remote, else target all
|
|
||
| | Service data attribute | Optional | Description | | ||
| | ---------------------- | -------- | ----------- | | ||
| | `entity_id` | yes | Only act on specific remote. Else targets all. |
There was a problem hiding this comment.
Missing determiner: on a specific remote
Use comma: remote, else target all
|
|
||
| | Service data attribute | Optional | Description | | ||
| | ---------------------- | -------- | ----------- | | ||
| | `entity_id` | yes | Only act on specific remote. Else targets all. |
There was a problem hiding this comment.
Missing determiner: on a specific remote
Use comma: remote, else target all
| | Service data attribute | Optional | Description | | ||
| | ---------------------- | -------- | ----------- | | ||
| | `entity_id` | yes | Only act on specific remote. Else targets all. | ||
| | `device` | no | Device ID to send command to. |
There was a problem hiding this comment.
Missing determiner: to send the command to
|
|
||
| | Service data attribute | Optional | Description | | ||
| | ---------------------- | -------- | ----------- | | ||
| | `entity_id` | yes | Only act on specific remote. Else targets all. |
There was a problem hiding this comment.
Missing determiner: on a specific remote
Use comma: remote, else target all
|
@amelchio Reviewed your PR and found a couple of tiny spelling/grammar things. Could you please take a look? |
|
@frenck Thanks. I fixed those now, also in the component doc where I had copied it from :) |
|
@amelchio Even better! Thank you for this 🏅 |
Description:
sync(moved to Harmony) anddevice(is now optional) from remote, remote.apple_tv and remote.itach.Checklist:
current. New documentation for platforms/components and features should go tonext.