-
-
Notifications
You must be signed in to change notification settings - Fork 695
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 latest docu of blockly features of M2 #2297
Conversation
Signed-off-by: Stefan Höhn <[email protected]>
✅ Thanks for your pull request to the openHAB documentation! The result can be previewed at the URL below (this comment and the preview will be updated if you add more commits).
To edit notification comments on pull requests, go to your Netlify site configuration. |
Signed-off-by: Stefan Höhn <[email protected]>
This pull request has been mentioned on openHAB Community. There might be relevant details there: https://community.openhab.org/t/openhab-4-2-milestone-discussion/154316/129 |
Signed-off-by: Stefan Höhn <[email protected]>
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.
In general LGTM, please have a look at my comments.
Please do not make the suggested changes locally and then push a commit, just go to the code tab, add the suggestions "to batch" and then "commit and sign-off batch" (green button). This automatically resolves the suggestions and eases review for us both 👍
Afterwards address the rest locally and push a commit please.
|
||
rules[“id”] gives you the rule object for the given id, which supports the [trigger method](https://openhab.github.io/openhab-jruby/main/OpenHAB/Core/Rules/Rule.html#trigger-instance_method) (aliased as run). | ||
|
||
> rules["scene_id"].run |
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.
Please use code fences and set ruby
or rb
as language.
Co-authored-by: Florian Hotze <[email protected]> Signed-off-by: stefan-hoehn <[email protected]>
Co-authored-by: Florian Hotze <[email protected]> Signed-off-by: stefan-hoehn <[email protected]>
Co-authored-by: Florian Hotze <[email protected]> Signed-off-by: stefan-hoehn <[email protected]>
Co-authored-by: Florian Hotze <[email protected]> Signed-off-by: stefan-hoehn <[email protected]>
Co-authored-by: Florian Hotze <[email protected]> Signed-off-by: stefan-hoehn <[email protected]>
Co-authored-by: Florian Hotze <[email protected]> Signed-off-by: stefan-hoehn <[email protected]>
Co-authored-by: Florian Hotze <[email protected]> Signed-off-by: stefan-hoehn <[email protected]>
Co-authored-by: Florian Hotze <[email protected]> Signed-off-by: stefan-hoehn <[email protected]>
Co-authored-by: Florian Hotze <[email protected]> Signed-off-by: stefan-hoehn <[email protected]>
Co-authored-by: Florian Hotze <[email protected]> Signed-off-by: stefan-hoehn <[email protected]>
Co-authored-by: Florian Hotze <[email protected]> Signed-off-by: stefan-hoehn <[email protected]>
Co-authored-by: Florian Hotze <[email protected]> Signed-off-by: stefan-hoehn <[email protected]>
Co-authored-by: Florian Hotze <[email protected]> Signed-off-by: stefan-hoehn <[email protected]>
Co-authored-by: Florian Hotze <[email protected]> Signed-off-by: stefan-hoehn <[email protected]>
Co-authored-by: Florian Hotze <[email protected]> Signed-off-by: stefan-hoehn <[email protected]>
Co-authored-by: Florian Hotze <[email protected]> Signed-off-by: stefan-hoehn <[email protected]>
Co-authored-by: Florian Hotze <[email protected]> Signed-off-by: stefan-hoehn <[email protected]>
Co-authored-by: Florian Hotze <[email protected]> Signed-off-by: stefan-hoehn <[email protected]>
Co-authored-by: Florian Hotze <[email protected]> Signed-off-by: stefan-hoehn <[email protected]>
Co-authored-by: Florian Hotze <[email protected]> Signed-off-by: stefan-hoehn <[email protected]>
Signed-off-by: Stefan Höhn <[email protected]>
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.
One last comment:
|
||
```ruby | ||
rules["scene_id"].run | ||
`` |
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.
There should be three of these signs.
Signed-off-by: Stefan Höhn <[email protected]>
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.
LGTM, thanks!
This adds missing documentation for the following new blockly features
and many more small corrections