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

sceneEngine.js: Introduce conditions & Refactorings #8

Merged
merged 6 commits into from May 13, 2022

Conversation

florian-h05
Copy link
Owner

@florian-h05 florian-h05 commented May 13, 2022

Additions

Introduce conditionFn which allows to use a expression to evaluate whether a scene target is valid now.

Breaking Changes

  • Create one scene engine rule per controller Item and not one rule for multiple (should increase performance as rule triggers are more fine grained).
  • Refactoring/Renaming of the sceneDefinition.

This changes have been successfully tested.

Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
@florian-h05 florian-h05 changed the title [WIP] sceneEngine.js: Introduce conditions & Refactor sceneEngine.js: Introduce conditions & Refactor May 13, 2022
@florian-h05 florian-h05 changed the title sceneEngine.js: Introduce conditions & Refactor sceneEngine.js: Introduce conditions & Refactorings May 13, 2022
@florian-h05 florian-h05 merged commit 69331be into main May 13, 2022
@florian-h05 florian-h05 deleted the sceneengine-refactoring branch May 13, 2022 22:47
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.

None yet

1 participant