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

Document the new dynamic scripting-language transformation #2042

Merged
merged 2 commits into from
Apr 16, 2023

Conversation

jimtng
Copy link
Contributor

@jimtng jimtng commented Mar 28, 2023

@netlify
Copy link

netlify bot commented Mar 28, 2023

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 cae0a1f
🔍 Latest deploy log https://app.netlify.com/sites/openhab-docs-preview/deploys/643bdb3c84a82f0008fc47a2
😎 Deploy Preview https://deploy-preview-2042--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 settings.

@jimtng jimtng force-pushed the script-transformation branch 6 times, most recently from 279385e to cdb81b4 Compare March 29, 2023 07:15
Signed-off-by: Jimmy Tanagra <jcode@tanagra.id.au>
@jimtng jimtng force-pushed the script-transformation branch from cdb81b4 to 2b99433 Compare March 29, 2023 07:19
@Confectrician Confectrician added the stat: dependency 💥 This issue/pr has a dependency in another repo label Apr 7, 2023
@jimtng
Copy link
Contributor Author

jimtng commented Apr 16, 2023

The feature has been merged, so this PR can also be merged, after a review.

Copy link
Contributor

@stefan-hoehn stefan-hoehn left a comment

Choose a reason for hiding this comment

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

Thanks for the amount of the work, you put it to this documentation.

addons/transformations.md Outdated Show resolved Hide resolved
addons/transformations.md Outdated Show resolved Hide resolved
addons/transformations.md Show resolved Hide resolved
@jimtng jimtng force-pushed the script-transformation branch from 546cdef to 63c3190 Compare April 16, 2023 11:25
Signed-off-by: Jimmy Tanagra <jcode@tanagra.id.au>
@jimtng jimtng force-pushed the script-transformation branch from 63c3190 to cae0a1f Compare April 16, 2023 11:25
Copy link
Contributor

@stefan-hoehn stefan-hoehn 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 added this to the 4.0 milestone Apr 16, 2023
@stefan-hoehn stefan-hoehn merged commit fd2b229 into openhab:main Apr 16, 2023
@jimtng jimtng deleted the script-transformation branch April 16, 2023 12:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stat: dependency 💥 This issue/pr has a dependency in another repo
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants