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

Add support for conditions #286

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

volfan6415
Copy link

This splits out the conditions work from pull request No. 219

- process any conditions listed as well
@5iver
Copy link
Member

5iver commented Dec 7, 2019

Thank you! Similar to the OH repos, I've been requiring sign-offs on commits... https://www.openhab.org/docs/developer/contributing.html#sign-your-work. You can either setup a local repo and use git to amend your commit, or close this PR and submit another (through the web) and include...

Signed-off-by: Joe Smith joe.smith@email.com

... at the bottom of the commit message.

volfan6415 added a commit to volfan6415/openhab-helper-libraries that referenced this pull request Feb 11, 2021
This code makes some changes to the rule generation which enables the use of the conditions.   I have been running this code for almost 2 years now without any issues.

I don't think this was implemented in the main fork but i made a pull requests on them.

see openhab-scripters#287

and  openhab-scripters#286 respectively.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants