-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
[danfossairunit] Fix already configured units in inbox #12217
[danfossairunit] Fix already configured units in inbox #12217
Conversation
Fixes openhab#12216 Signed-off-by: Jacob Laursen <jacob-github@vindvejr.dk>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This was already set correctly by the discovery class, so need to other change than what you proposed. |
Not sure what you mean? You are right about discovery class, so should be in sync now:
|
Yes, I was just mentioning your change in the XML file was sufficient. |
Fixes openhab#12216 Signed-off-by: Jacob Laursen <jacob-github@vindvejr.dk>
Fixes openhab#12216 Signed-off-by: Jacob Laursen <jacob-github@vindvejr.dk> Signed-off-by: Nick Waterton <n.waterton@outlook.com>
Fixes openhab#12216 Signed-off-by: Jacob Laursen <jacob-github@vindvejr.dk>
Fixes openhab#12216 Signed-off-by: Jacob Laursen <jacob-github@vindvejr.dk>
Fixes openhab#12216 Signed-off-by: Jacob Laursen <jacob-github@vindvejr.dk> Signed-off-by: Andras Uhrin <andras.uhrin@gmail.com>
Fixes openhab#12216 Signed-off-by: Jacob Laursen <jacob-github@vindvejr.dk>
Fixes #12216
Signed-off-by: Jacob Laursen jacob-github@vindvejr.dk
Fix entries in inbox for already configured units.