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 latest docu of blockly features of M2 #2297

Merged
merged 26 commits into from
May 27, 2024

Conversation

stefan-hoehn
Copy link
Contributor

@stefan-hoehn stefan-hoehn added this to the 4.2 milestone May 19, 2024
Copy link

netlify bot commented May 19, 2024

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).

Name Link
🔨 Latest commit 3b6a227
🔍 Latest deploy log https://app.netlify.com/sites/openhab-docs-preview/deploys/6654c225c617040007f39f32
😎 Deploy Preview https://deploy-preview-2297--openhab-docs-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Signed-off-by: Stefan Höhn <[email protected]>
@openhab-bot
Copy link
Collaborator

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]>
Copy link
Contributor

@florian-h05 florian-h05 left a 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.

configuration/blockly/index.md Outdated Show resolved Hide resolved
configuration/blockly/index.md Outdated Show resolved Hide resolved
configuration/blockly/index.md Outdated Show resolved Hide resolved
configuration/blockly/index.md Outdated Show resolved Hide resolved
configuration/blockly/index.md Outdated Show resolved Hide resolved

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
Copy link
Contributor

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.

configuration/blockly/rules-blockly-timers-and-delays.md Outdated Show resolved Hide resolved
configuration/blockly/rules-blockly-timers-and-delays.md Outdated Show resolved Hide resolved
configuration/blockly/rules-blockly-timers-and-delays.md Outdated Show resolved Hide resolved
configuration/blockly/rules-blockly-timers-and-delays.md Outdated Show resolved Hide resolved
stefan-hoehn and others added 21 commits May 27, 2024 12:58
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]>
Copy link
Contributor

@florian-h05 florian-h05 left a 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
``
Copy link
Contributor

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]>
Copy link
Contributor

@florian-h05 florian-h05 left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@stefan-hoehn stefan-hoehn merged commit a3accd8 into openhab:main May 27, 2024
5 checks passed
@stefan-hoehn stefan-hoehn deleted the blockly_updates branch May 27, 2024 18:02
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.

3 participants