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

[miio] channels removal #9237

Merged
merged 1 commit into from
Dec 7, 2020
Merged

Conversation

marcelrv
Copy link
Contributor

@marcelrv marcelrv commented Dec 5, 2020

This ensures channels are removed if you switch between various models, while preserving the standard channels available to all models

Signed-off-by: Marcel Verpaalen marcel@verpaalen.com

Signed-off-by: Marcel Verpaalen <marcel@verpaalen.com>
@marcelrv marcelrv requested a review from cpmeister December 5, 2020 09:13
@marcelrv
Copy link
Contributor Author

marcelrv commented Dec 5, 2020

@cpmeister wrt to the channels not being available in your binding,

I see the ChannelProvider being queried when I do the check for the availability of the channel https://github.com/openhab/openhab-addons/blob/main/bundles/org.openhab.binding.miio/src/main/java/org/openhab/binding/miio/internal/handler/MiIoBasicHandler.java#L468 maybe that is causing a refresh of the available channels in the framework and the reason I did not have similar issue

@cpmeister cpmeister merged commit ab3dcba into openhab:main Dec 7, 2020
@cpmeister cpmeister added the enhancement An enhancement or new feature for an existing add-on label Dec 7, 2020
@cpmeister cpmeister added this to the 3.0.0.M6 milestone Dec 7, 2020
@marcelrv marcelrv deleted the miio-channelClean branch December 7, 2020 14:04
chrisonline pushed a commit to chrisonline/openhab-addons that referenced this pull request Dec 7, 2020
Signed-off-by: Marcel Verpaalen <marcel@verpaalen.com>
Signed-off-by: Christian Grasser <info@christiangrasser.at>
boehan pushed a commit to boehan/openhab-addons that referenced this pull request Apr 12, 2021
Signed-off-by: Marcel Verpaalen <marcel@verpaalen.com>
marcfischerboschio pushed a commit to bosch-io/openhab-addons that referenced this pull request May 5, 2022
Signed-off-by: Marcel Verpaalen <marcel@verpaalen.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.

2 participants