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

Instead of a single event for NodeExecution, add NodeExecutionStart and NodeExecutionEnd #176

Merged
merged 2 commits into from
Dec 1, 2022

Conversation

bhouston
Copy link
Owner

@bhouston bhouston commented Dec 1, 2022

This can enable: #160

Because node input values will be updated upon NodeExecuteStart.

@bhouston bhouston merged commit 5bafbca into main Dec 1, 2022
@bhouston bhouston deleted the start-stop-events branch December 6, 2022 21:43
@oveddan
Copy link
Collaborator

oveddan commented Dec 7, 2022

Would be nice to get a chance to review this PR before it's merged, to give an opportunity to others to provide feedback/ask questions.

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.

2 participants