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

[hpprinter] Change status channel from hash map values to state description options #11739

Merged
merged 1 commit into from
Dec 9, 2021

Conversation

Cossey
Copy link
Contributor

@Cossey Cossey commented Dec 9, 2021

This changes how both the status channels work. Originally, they referenced a hash table with the name of the status value. It seems more appropriate to have these as state description options as it would allow the statuses to be localised.

Signed-off-by: Stewart Cossey <stewart.cossey@gmail.com>
@Cossey Cossey self-assigned this Dec 9, 2021
@Cossey Cossey added the enhancement An enhancement or new feature for an existing add-on label Dec 9, 2021
@Cossey Cossey requested a review from a team December 9, 2021 07:39
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

@lolodomo lolodomo merged commit dc11890 into openhab:main Dec 9, 2021
@lolodomo lolodomo added this to the 3.2 milestone Dec 9, 2021
@Cossey Cossey deleted the fix-hpprinter branch December 9, 2021 08:17
@wborn wborn changed the title [hpprinter] Change status channel from hash map values to state description options. [hpprinter] Change status channel from hash map values to state description options Dec 18, 2021
NickWaterton pushed a commit to NickWaterton/openhab-addons that referenced this pull request Dec 30, 2021
…ns. (openhab#11739)

Signed-off-by: Stewart Cossey <stewart.cossey@gmail.com>
Signed-off-by: Nick Waterton <n.waterton@outlook.com>
mischmidt83 pushed a commit to mischmidt83/openhab-addons that referenced this pull request Jan 9, 2022
…ns. (openhab#11739)

Signed-off-by: Stewart Cossey <stewart.cossey@gmail.com>
Signed-off-by: Michael Schmidt <mi.schmidt.83@gmail.com>
nemerdaud pushed a commit to nemerdaud/openhab-addons that referenced this pull request Jan 28, 2022
…ns. (openhab#11739)

Signed-off-by: Stewart Cossey <stewart.cossey@gmail.com>
volkmarnissen pushed a commit to volkmarnissen/openhab-addons that referenced this pull request Mar 3, 2022
…ns. (openhab#11739)

Signed-off-by: Stewart Cossey <stewart.cossey@gmail.com>
marcfischerboschio pushed a commit to bosch-io/openhab-addons that referenced this pull request May 5, 2022
…ns. (openhab#11739)

Signed-off-by: Stewart Cossey <stewart.cossey@gmail.com>
andan67 pushed a commit to andan67/openhab-addons that referenced this pull request Nov 6, 2022
…ns. (openhab#11739)

Signed-off-by: Stewart Cossey <stewart.cossey@gmail.com>
andrasU pushed a commit to andrasU/openhab-addons that referenced this pull request Nov 12, 2022
…ns. (openhab#11739)

Signed-off-by: Stewart Cossey <stewart.cossey@gmail.com>
Signed-off-by: Andras Uhrin <andras.uhrin@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.

None yet

2 participants