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

[nuvo] Include NuvoNet source favorites in zone favorite channel #15292

Merged
merged 4 commits into from
Jul 24, 2023

Conversation

mlobstein
Copy link
Contributor

The main goal of this PR is to accomplish a minor refactoring by standardizing the use of the Zone/Source enum instead of numeric string keys that were inconsistently implemented.

One small new feature is included that improves the utility of the favorite channel in each zone. In additional to the 12 built-in zone favorites, any favorites specified by the user for an openHAB NuvoNet source (visible in the physical keypad) are also displayed and can be selected by the favorite channel.

Signed-off-by: Michael Lobstein <michael.lobstein@gmail.com>
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 Jul 22, 2023
Signed-off-by: Michael Lobstein <michael.lobstein@gmail.com>
@mlobstein mlobstein added rebuild Triggers Jenkins PR build and removed rebuild Triggers Jenkins PR build labels Jul 24, 2023
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, LGTM! Only one suggestion to use Stream.toList() introduced in Java 16.

Signed-off-by: Michael Lobstein <michael.lobstein@gmail.com>
@jlaur jlaur merged commit 51f3bf6 into openhab:main Jul 24, 2023
3 checks passed
@jlaur jlaur added this to the 4.1 milestone Jul 24, 2023
@mlobstein mlobstein deleted the nuvo_favorites branch July 24, 2023 23:56
Pshatsillo pushed a commit to Pshatsillo/openhab-addons that referenced this pull request Sep 29, 2023
…penhab#15292)

* Include NuvoNet source favorites in zone `favorite` channel
* Use existing constants
* VALID_SOURCES to enum list

Signed-off-by: Michael Lobstein <michael.lobstein@gmail.com>
austvik pushed a commit to austvik/openhab-addons that referenced this pull request Mar 27, 2024
…penhab#15292)

* Include NuvoNet source favorites in zone `favorite` channel
* Use existing constants
* VALID_SOURCES to enum list

Signed-off-by: Michael Lobstein <michael.lobstein@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.

None yet

2 participants