-
Notifications
You must be signed in to change notification settings - Fork 15
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
Feature request: Add support for all rule actions #82
Comments
Shouldn't this be already generated within JRuleActions? |
All my generated actions are ThingActions, but I am talking about the more generic "module" actions. They are at least not available in the generated code on my side. |
I never used this, thought these are actions from the myopenhab binding. |
I guess it boils down to which addons that a user has installed, so I believe we need to generate this as well. |
I think this is contained in the openhab "core", because it's documented here: https://www.openhab.org/docs/configuration/actions.html#cloud-notification-actions |
Pretty sure you need to install the openhabcloud addon to actually have it available. This is the impl; https://github.com/openhab/openhab-addons/blob/0154427c99e5f55dd201d74d079e98afe26fcaab/bundles/org.openhab.io.openhabcloud/src/main/java/org/openhab/io/openhabcloud/NotificationAction.java This is the base Handler class for the various actions they provide; https://github.com/openhab/openhab-addons/blob/0154427c99e5f55dd201d74d079e98afe26fcaab/bundles/org.openhab.io.openhabcloud/src/main/java/org/openhab/io/openhabcloud/internal/actions/BaseNotificationActionHandler.java#L28 |
Yes, you're right. So it makes sense to generate this as well. |
Currently we have support for thing actions which is great.
But I'd also love to see support for other actions like the openhab cloud sendNotification action.
Could be generated code just like ThingActions today.
WDYT @querdenker2k ?
The text was updated successfully, but these errors were encountered: