这是indexloc提供的服务,不要输入任何密码
Skip to content
This repository was archived by the owner on May 17, 2021. It is now read-only.

Conversation

@teichsta
Copy link
Member

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

@teichsta
Copy link
Member Author

unfortunately my.openHAB Action is still not working properly

Copy link
Member

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.

Copy link
Member Author

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?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

see issue #2969

@gerrieg
Copy link
Contributor

gerrieg commented Jul 27, 2015

The homematic action requires a correctly configured homematic binding (no item).
The astro action does not require a correctly configured binding, but uses the calculation methods in the binding.

If a action should not reference a binding, what's the correct way to implement this?

@teichsta teichsta force-pushed the fix-missing-actions-in-designer branch from 87a904d to f5d0411 Compare July 27, 2015 12:47
…pendencies in order to activate them as soon as classes are needed

Signed-off-by: Thomas Eichstädt-Engelen <thomas@openhab.org>
@teichsta teichsta force-pushed the fix-missing-actions-in-designer branch from f5d0411 to 9cb021a Compare July 29, 2015 22:29
teichsta added a commit that referenced this pull request Jul 31, 2015
@teichsta teichsta merged commit 8138819 into openhab:master Jul 31, 2015
@teichsta teichsta deleted the fix-missing-actions-in-designer branch July 31, 2015 19:07
watou added a commit to watou/openhab that referenced this pull request Aug 1, 2015
…Designer; moved messages package out of internal; made ecobeeSetHold temp parameters truly optional.
@lolodomo
Copy link
Contributor

lolodomo commented Aug 1, 2015

I can confirm MiOS action is now visible in Designer 1.7.1.
Thank you

@udo1toni
Copy link
Contributor

udo1toni commented Aug 2, 2015

I can confirm Squeezebox action is now visible in Designer 1.7.1
Thanks :)

@watou watou mentioned this pull request Aug 21, 2015
watou added a commit to watou/openhab that referenced this pull request Aug 23, 2015
* 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.
watou added a commit to watou/openhab that referenced this pull request Aug 26, 2015
* 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.
paolodenti pushed a commit to paolodenti/openhab that referenced this pull request Sep 28, 2015
* 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.
paolodenti pushed a commit to paolodenti/openhab that referenced this pull request Oct 1, 2015
* 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.
@watou watou mentioned this pull request Oct 11, 2015
@watou watou mentioned this pull request Dec 4, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants