elsa-workflows/elsa-core

IUserTaskService enhancement - optional activityId

オープン

#4,535 opened on 2023/10/13

 (3 件のコメント) (0 件のリアクション) (0 人の担当者)C# (979 件のフォーク)batch import
bugelsa 2elsa 3enhancementgood first issuetriaged

Repository metrics

Stars
 (5,486 個のスター)
PR merge metrics
 (平均マージ 1d 17h) (30d で 29 merged PRs)

説明

The IUserTaskService is great but, I think it needs an improvement.

Specifically, i think the TriggerUserAction record needs an optional activityId, to be absolutely specific if the action being performed is going to target the correct activity.

public record TriggerUserAction(string Action,string? WorkflowInstanceId = default,string? CorrelationId = default );

to become

public record TriggerUserAction(string Action,string? WorkflowInstanceId = default, string? ActivityId = default, string? CorrelationId = default );

If you consider the possibility that a Fork has a blocking activity on each branch, and each perhaps has the same outcome names, then the specificity requires in that use providing the activityId.

This change would also predicate the IUserTaskService needing two additional methods:

        Task< CollectedWorkflow> ExecuteUserActionAsync(TriggerUserAction taskAction, CancellationToken cancellationToken = default(CancellationToken));

        Task<CollectedWorkflow> DispatchUserActionAsync(TriggerUserAction taskAction, CancellationToken cancellationToken = default(CancellationToken));

コントリビューターガイド