这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@morph166955
Copy link
Contributor

Adds the ability on non-GoogleTV things to disable the GoogleTV protocol stack. This is in preparation for addition of new protocol stacks coming soon.

Signed-off-by: Ben Rosenblum <rosenblumb@gmail.com>
@jlaur
Copy link
Contributor

jlaur commented Jul 29, 2023

@morph166955 - two things:

  • Can you add this to the README?
  • Since it's not yet documented, I need to ask: When would the user disable this? Is it impossible to auto-detect when this can be disabled?

@morph166955
Copy link
Contributor Author

I can absolutely add this to the readme.

This is in preparation for adding the Phillips and Sony protocols to the binding which I'm currently working. Philips it's no longer maintained so I'm trying to integrate here both because it makes sense and to support the community going forward. Sony is maintained in a maintenance mode, and really needs to get merged in as well. I don't anticipate or encourage users to disable Google regularly. However, for testing, or to maintain backwards compatibility, the may choose to use that protocol instead.

Signed-off-by: Ben Rosenblum <rosenblumb@gmail.com>
@morph166955
Copy link
Contributor Author

@wborn null check removed as requested. I'm going to do another PR when I get a few minutes to remove the rest from the existing code.

@jlaur README updated as requested

@lolodomo lolodomo added the enhancement An enhancement or new feature for an existing add-on label Jul 30, 2023
@morph166955
Copy link
Contributor Author

Is there anything else needed on this?

@morph166955
Copy link
Contributor Author

I'm going to have to rebase. Looks like the i18n got out of sync.

@morph166955 morph166955 force-pushed the androidtv-gtvoption branch from 92ecb11 to 43e60a2 Compare July 31, 2023 15:43
morph166955 and others added 2 commits July 31, 2023 10:50
Signed-off-by: Ben Rosenblum <rosenblumb@gmail.com>
@morph166955
Copy link
Contributor Author

That should resolve the conflict.

Copy link
Contributor

@lolodomo lolodomo left a comment

Choose a reason for hiding this comment

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

LGTM, thank you

@lolodomo lolodomo merged commit 3e1686b into openhab:main Jul 31, 2023
@lolodomo lolodomo added this to the 4.1 milestone Jul 31, 2023
@morph166955 morph166955 deleted the androidtv-gtvoption branch July 31, 2023 21:38
Pshatsillo pushed a commit to Pshatsillo/openhab-addons that referenced this pull request Sep 29, 2023
* Adds gtvEnabled

---------

Signed-off-by: Ben Rosenblum <rosenblumb@gmail.com>
austvik pushed a commit to austvik/openhab-addons that referenced this pull request Mar 27, 2024
* Adds gtvEnabled

---------

Signed-off-by: Ben Rosenblum <rosenblumb@gmail.com>
Signed-off-by: Jørgen Austvik <jaustvik@acm.org>
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.

4 participants