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

Update WorkflowInstanceDetails with ActivityExecutionRecord #168

Merged
merged 3 commits into from Mar 8, 2024

Conversation

sfmskywalker
Copy link
Member

Refactored WorkflowInstanceDetails in Elsa.Studio.Workflows to use ActivityExecutionRecord instead of ActivityExecutionContextState. This involved changing variable references, adjusting async methods, and switching from a package reference to a project reference for Elsa.Api.Client in Elsa.Studio.Core.

Refactored WorkflowInstanceDetails in Elsa.Studio.Workflows to use ActivityExecutionRecord instead of ActivityExecutionContextState. This involved changing variable references, adjusting async methods, and switching from a package reference to a project reference for Elsa.Api.Client in Elsa.Studio.Core.
Removed commented out code and unnecessary default initializations in WorkflowInstanceDetails.razor.cs. This includes the commented out _workflowActivityExecutionContext and unnecessary default initializations for SelectedSubWorkflow and SelectedSubWorkflowExecutions. The refactoring increases code readability and efficiency.
The Elsa.Api.Client version has been updated from 3.1.0-preview.1152 to 3.1.0-preview.1163 in the Directory.Packages.props file. Additionally, the reference method to the Elsa.Api.Client in the Elsa.Studio.Core project has been changed from a direct project reference to a package reference.
@sfmskywalker sfmskywalker merged commit 1c8911d into main Mar 8, 2024
6 checks passed
@sfmskywalker sfmskywalker deleted the issue/5038 branch March 8, 2024 08:24
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

2 participants