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

[kodi] Changed 'systemcommand' channel to command options #7369

Merged
merged 1 commit into from
Sep 13, 2020

Conversation

cweitkamp
Copy link
Contributor

  • Changed 'systemcommand' channel to command options

Testing for openhab/openhab-core#1422 and openhab/openhab-webui#217

Signed-off-by: Christoph Weitkamp github@christophweitkamp.de

@cweitkamp cweitkamp added enhancement An enhancement or new feature for an existing add-on work in progress A PR that is not yet ready to be merged labels Apr 14, 2020
@cweitkamp cweitkamp requested a review from pail23 as a code owner April 14, 2020 10:19
@Hilbrand
Copy link
Member

@cweitkamp What are your plans for this pull request? The code looks ok. Except for the current conflict does it need any changes?

@cweitkamp
Copy link
Contributor Author

@Hilbrand in general this implementation is final. I will try to solve the conflicts asap to get this PR into mergable status.

Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de>
@cweitkamp
Copy link
Contributor Author

@Hilbrand Rebase done.

Copy link
Member

@Hilbrand Hilbrand left a comment

Choose a reason for hiding this comment

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

LGTM. I assume the label work-in-progress can be removed?

@cweitkamp cweitkamp removed the work in progress A PR that is not yet ready to be merged label Sep 13, 2020
@cweitkamp cweitkamp changed the title [WIP][kodi] Changed 'systemcommand' channel to command options [kodi] Changed 'systemcommand' channel to command options Sep 13, 2020
@cweitkamp
Copy link
Contributor Author

Yes, of course. I removed it.

@Hilbrand Hilbrand merged commit 186da70 into openhab:2.5.x Sep 13, 2020
@Hilbrand Hilbrand added this to the 2.5.9 milestone Sep 13, 2020
@cweitkamp cweitkamp deleted the feature-kodi-command-options branch September 13, 2020 16:20
markus7017 pushed a commit to markus7017/openhab-addons that referenced this pull request Sep 19, 2020
Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de>
andrewfg pushed a commit to andrewfg/openhab-addons that referenced this pull request Oct 8, 2020
Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de>
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.

2 participants