-
Notifications
You must be signed in to change notification settings - Fork 726
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
[Workflow] Introduce human interaction workflow pattern #3446
Conversation
Signed-off-by: Chris Gillum <cgillum@microsoft.com>
Signed-off-by: Chris Gillum <cgillum@microsoft.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My comments. Can you ensure that the diagram is included in the diagrams PPT file. @hhunter-ms can do this for you, if you give her the diagram source
daprdocs/content/en/developing-applications/building-blocks/workflow/workflow-patterns.md
Outdated
Show resolved
Hide resolved
daprdocs/content/en/developing-applications/building-blocks/workflow/workflow-patterns.md
Outdated
Show resolved
Hide resolved
daprdocs/content/en/developing-applications/building-blocks/workflow/workflow-patterns.md
Outdated
Show resolved
Hide resolved
daprdocs/content/en/developing-applications/building-blocks/workflow/workflow-patterns.md
Outdated
Show resolved
Hide resolved
daprdocs/content/en/developing-applications/building-blocks/workflow/workflow-patterns.md
Outdated
Show resolved
Hide resolved
daprdocs/content/en/developing-applications/building-blocks/workflow/workflow-patterns.md
Outdated
Show resolved
Hide resolved
daprdocs/content/en/developing-applications/building-blocks/workflow/workflow-patterns.md
Outdated
Show resolved
Hide resolved
daprdocs/content/en/developing-applications/building-blocks/workflow/workflow-patterns.md
Outdated
Show resolved
Hide resolved
daprdocs/content/en/developing-applications/building-blocks/workflow/workflow-patterns.md
Outdated
Show resolved
Hide resolved
daprdocs/content/en/developing-applications/building-blocks/workflow/workflow-patterns.md
Outdated
Show resolved
Hide resolved
Stale PR, paging all reviewers |
Signed-off-by: Chris Gillum <cgillum@microsoft.com>
…docs into human-interaction2
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Description
Starting in v1.11, we're introducing the human interaction pattern for workflows that takes advantage of the new "raise event" workflow API. This PR updates the workflow patterns documentation to include a description of this pattern and provide some brief code examples using the Dapr SDKs.
This PR only includes code snippets for C#. Once available, we'll need to also add Python code snippets (this whole page will need Python snippets).
@hhunter-ms I've added a starter diagram to this PR, but can you replace it with something more official looking?(Image has been updated in the latest commit)
Issue reference
Part of #3350.