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 input application to workflow state extractor #4829

Merged
merged 2 commits into from Jan 23, 2024
Merged

Conversation

sfmskywalker
Copy link
Member

A new method ApplyInput has been added to prevent overwriting any existing input in the WorkflowExecutionContext. Now, input items are added to the workflow execution context only if they don't exist there already. The Apply method has been updated as well to add workflow state inputs appropriately.

Fixes #4705

A new method ApplyInput has been added to prevent overwriting any existing input in the WorkflowExecutionContext. Now, input items are added to the workflow execution context only if they don't exist there already. The Apply method has been updated as well to add workflow state inputs appropriately.

Fixes #4705
@sfmskywalker sfmskywalker added the elsa 3 This issue is specific to Elsa 3 label Jan 23, 2024
The private methods within the WorkflowStateExtractor service have been reordered for readability and maintainability. The `ApplyInput` and `GetPersistableInput` methods have been moved to a position just before the `ExtractProperties` method. This ensures that similar or related methods are grouped together, improving the logical structure of the code.
@sfmskywalker sfmskywalker added this to the Elsa 3.1 milestone Jan 23, 2024
@sfmskywalker sfmskywalker merged commit a1b7afd into v3.0.4 Jan 23, 2024
1 check passed
@sfmskywalker sfmskywalker deleted the fix(4705) branch January 23, 2024 21:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
elsa 3 This issue is specific to Elsa 3
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

1 participant