Skip to content
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

[roku] Add End Time and Media Progress channels #18059

Merged
merged 2 commits into from
Jan 9, 2025

Conversation

mlobstein
Copy link
Contributor

This PR adds the following read-only channels to the binding similar to the Plex binding :

  • End Time: The date/time when the currently playing media will end.
  • Media Progress: A dimmer that shows the current progress [0-100%] of the playing media.

The values for these channels are calculated from the Elapsed Time and Total Time values provided by the Roku API. Some streaming apps do not provide these values and in that case these channels will display an UNDEF value.

Signed-off-by: Michael Lobstein <michael.lobstein@gmail.com>
@mlobstein mlobstein added the enhancement An enhancement or new feature for an existing add-on label Jan 7, 2025
@mlobstein mlobstein requested a review from morph166955 January 7, 2025 03:51
@mlobstein mlobstein changed the title [Roku] Add End Time and Media Progress channels [roku] Add End Time and Media Progress channels Jan 7, 2025
morph166955

This comment was marked as duplicate.

Copy link
Contributor

@morph166955 morph166955 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@jlaur jlaur left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the new channels. Can you also regenerate the I18N properties file since new channel types were added? Otherwise LGTM.

Signed-off-by: Michael Lobstein <michael.lobstein@gmail.com>
Copy link
Contributor

@jlaur jlaur left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jlaur jlaur merged commit f6efa87 into openhab:main Jan 9, 2025
2 checks passed
@jlaur jlaur added this to the 5.0 milestone Jan 9, 2025
@mlobstein mlobstein deleted the roku_endtime branch January 9, 2025 15:24
GearrelW pushed a commit to GearrelW/openhab-addons that referenced this pull request Jan 12, 2025
Signed-off-by: Michael Lobstein <michael.lobstein@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An enhancement or new feature for an existing add-on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants