Skip to content
This repository has been archived by the owner on Sep 6, 2023. It is now read-only.

docs: document UI created rules #151

Merged
merged 1 commit into from
Feb 11, 2021

Conversation

pacive
Copy link
Collaborator

@pacive pacive commented Feb 10, 2021

Just a quick write-up on how to create rules in the UI, to avoid confusion from users taking that path. Note that one of the examples requires that #134 is merged, so might want to hold off this until then. Feel free to add more examples!

Signed-off-by: Anders Alfredsson andersb86@gmail.com

Signed-off-by: Anders Alfredsson <andersb86@gmail.com>
@boc-tothefuture
Copy link
Owner

boc-tothefuture commented Feb 10, 2021

Awesome! Thanks for the doc.
I don't think this actually closes #13.

#13 needs to describe the fact that there are really "two passes" in this system. There is a pass that creates the rules when the rules file is read by openhab. Then later the rules get executed by the system and they are distinct. So anything outside of rule block gets executed when the file is parsed by OpenHAB.

It is somewhat analogous to the chef two pass model

@pacive
Copy link
Collaborator Author

pacive commented Feb 10, 2021

Ah, can remove the link to the issue so it won't be closed. Do you think that should be explained in more detail here or elsewhere in the docs? Should perhaps go in the usage/index.md?

@boc-tothefuture
Copy link
Owner

I think it should go in usage/index.md.

This is a welcome addition as it is for those using the UI.. Thanks for the PR!

@boc-tothefuture boc-tothefuture merged commit a6451b9 into boc-tothefuture:main Feb 11, 2021
@github-actions
Copy link

🎉 This PR is included in version 2.16.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

ccutrer referenced this pull request in ccutrer/openhab-jrubyscripting Dec 27, 2021
docs: document UI created rules
Former-commit-id: a6451b9
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants