-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
fix for #1403 - makes Actions visible in Designer #2967
fix for #1403 - makes Actions visible in Designer #2967
Conversation
e2f826e to
87a904d
Compare
|
unfortunately my.openHAB Action is still not working properly |
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.
A bundle must not import internal packages from other bundles.
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.
Absolutely right and we should change this (out of the scope of this Bugfix). What i changed was simply removing the require-bundle deep and add the import-package instead. @gerrieg could you please take care of this and create another PR moving the externally required classes in the appropriate package?
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.
see issue #2969
|
The homematic action requires a correctly configured homematic binding (no item). If a action should not reference a binding, what's the correct way to implement this? |
87a904d to
f5d0411
Compare
…pendencies in order to activate them as soon as classes are needed Signed-off-by: Thomas Eichstädt-Engelen <thomas@openhab.org>
f5d0411 to
9cb021a
Compare
fix for #1403 - make Actions visible in Designer
…Designer; moved messages package out of internal; made ecobeeSetHold temp parameters truly optional.
|
I can confirm MiOS action is now visible in Designer 1.7.1. |
|
I can confirm Squeezebox action is now visible in Designer 1.7.1 |
* mult-ecobees-1-acct-bug (PR openhab#2765) * ecobee-celsius-arithmetic-exception (openhab#2793) * ecobee-discard-tokens (PR openhab#2849) * ecobee-echo-cancellation (PR openhab#2942) * changes similar to @teichsta's PR openhab#2967 so actions now appear in Designer; moved messages package out of internal; made ecobeeSetHold temp paramter truly optional.
* mult-ecobees-1-acct-bug (PR openhab#2765) * ecobee-celsius-arithmetic-exception (openhab#2793) * ecobee-discard-tokens (PR openhab#2849) * ecobee-echo-cancellation (PR openhab#2942) * changes similar to @teichsta's PR openhab#2967 so actions now appear in Designer; moved messages package out of internal; made ecobeeSetHold temp paramter truly optional. * Quiesce logging when network failure keeps a poll from completing.
* mult-ecobees-1-acct-bug (PR openhab#2765) * ecobee-celsius-arithmetic-exception (openhab#2793) * ecobee-discard-tokens (PR openhab#2849) * ecobee-echo-cancellation (PR openhab#2942) * changes similar to @teichsta's PR openhab#2967 so actions now appear in Designer; moved messages package out of internal; made ecobeeSetHold temp paramter truly optional. * Quiesce logging when network failure keeps a poll from completing.
* mult-ecobees-1-acct-bug (PR openhab#2765) * ecobee-celsius-arithmetic-exception (openhab#2793) * ecobee-discard-tokens (PR openhab#2849) * ecobee-echo-cancellation (PR openhab#2942) * changes similar to @teichsta's PR openhab#2967 so actions now appear in Designer; moved messages package out of internal; made ecobeeSetHold temp paramter truly optional. * Quiesce logging when network failure keeps a poll from completing.
especially added bundle activationpolicy to dependencies in order to activate them as soon as classes are needed
Signed-off-by: Thomas Eichstädt-Engelen thomas@openhab.org