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

Auto-refresh of Activity Execution Record Details #301

Open
wants to merge 9 commits into
base: blueberry
Choose a base branch
from

Conversation

sfmskywalker
Copy link
Member

No description provided.

Introduced the ActivityExecutionRecordSummary model and related API endpoints to support listing and retrieving activity execution summaries. Updated the workflow instance viewer and remote service to utilize the new summaries and provide detailed activity execution records on demand.
Introduced periodic refresh of activity execution records to ensure all details are fetched and updated appropriately. Added `IsFused` extension method to determine if the activity execution record has all necessary details. Implemented timers to manage the periodic refresh and dispose of resources accordingly.
Replace synchronous Refresh method with async version in ActivityExecutionsTab. Enhanced the periodic refresh handling with improved null checks and proper disposal. This ensures better resource management and correctness of refresh operations.
Eliminated unnecessary comments regarding the checking and periodic refreshing of selected item details. This change improves code readability and maintains the focus on the essential logic.
Renamed all instances of `ActivityExecution` to `ActivityExecutionSummary` for better clarity. Updated corresponding parameters, models, and service references in the affected components.
Clarified that LastActivityExecution represents the latest activity execution record. This helps in understanding that it is used for displaying the last state of the activity.
Base automatically changed from enh/activity-executions-lazy-load to blueberry October 11, 2024 09:57
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.

1 participant