-
-
Notifications
You must be signed in to change notification settings - Fork 429
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
Introduce a profile for the generic SCRIPT transformation #3292
Conversation
a625a3b
to
bc2da59
Compare
This pull request has been mentioned on openHAB Community. There might be relevant details there: https://community.openhab.org/t/openhab-4-0-snapshot-discussion/142322/93 |
Can you have another look at this @kaikreuzer? |
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.
Thanks, looks pretty good to me.
Could you please rebase the branch in order to resolve the conflict?
Just a few small comments below for your consideration.
...ipt/src/main/java/org/openhab/core/automation/module/script/ScriptTransformationService.java
Outdated
Show resolved
Hide resolved
...le.script/src/main/java/org/openhab/core/automation/module/script/profile/ScriptProfile.java
Show resolved
Hide resolved
...le.script/src/main/java/org/openhab/core/automation/module/script/profile/ScriptProfile.java
Show resolved
Hide resolved
...le.script/src/main/java/org/openhab/core/automation/module/script/profile/ScriptProfile.java
Outdated
Show resolved
Hide resolved
...le.script/src/main/java/org/openhab/core/automation/module/script/profile/ScriptProfile.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Jan N. Klug <github@klug.nrw>
bc2da59
to
16a480f
Compare
Signed-off-by: Jan N. Klug <github@klug.nrw>
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.
Thanks!
* Introduce a generic script profile Signed-off-by: Jan N. Klug <github@klug.nrw> GitOrigin-RevId: 7bad9ba
This is a replacement for #2872 which for unknown reasons can't be re-opened. I did another check and found that a general TRANSFORM profile is not feasible without a nearly complete refactoring of the whole transformation infrastructure. It would also need extensive refactoring in the UI part which I can't do.