Skip to content

Commit

Permalink
fix: Remove todos
Browse files Browse the repository at this point in the history
  • Loading branch information
jlacivita committed Mar 12, 2024
1 parent c3c2122 commit dc434bb
Showing 1 changed file with 0 additions and 8 deletions.
8 changes: 0 additions & 8 deletions requirements/specifications/intents/command-and-control.md
Original file line number Diff line number Diff line change
Expand Up @@ -200,8 +200,6 @@ optional relative field:
The value is a positive or negative integer that is relative to a scale
of 0-100.

**TODO**: Firebolt currently specifies the value as 0-1

Firebolt will not support complicated relative changes, e.g. "Set the
volume to 50% *of what it currently is\...*"

Expand Down Expand Up @@ -325,8 +323,6 @@ Media:
}
```

**TODO**: this is different than what we officially published. We need to track down if anyone implemented the old Pause Intent. If so, we need to support pause w/ value: false to mean resume.

If the action is pause, then the currently playing media should be
paused, with the frames on-screen and the video decoder ready to resume.

Expand Down Expand Up @@ -442,8 +438,6 @@ These intents manipulate accessibility features on the device.

This intent allows a user to turn closed captions on or off.

**TODO**: there is no hyphen in closed captions because it was already spec'd w/out one previously. Need to check if any platforms implemented it or not.

```json
{
"type": "xrn:firebolt:intent:platform:accessibility",
Expand Down Expand Up @@ -620,8 +614,6 @@ Finally, the intent **MAY** specify a `verbosity` property, which **MUST** use o

This intent allows a user to turn audio descriptions of content on or off.

**TODO**: there is no hyphen in closed captions because it was already spec'd w/out one previously. Need to check if any platforms implemented it or not.


```json
{
Expand Down

0 comments on commit dc434bb

Please sign in to comment.