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

[Workflow] Introduce human interaction workflow pattern #3446

Merged
merged 7 commits into from
May 31, 2023

Conversation

cgillum
Copy link
Contributor

@cgillum cgillum commented May 22, 2023

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.

Signed-off-by: Chris Gillum <cgillum@microsoft.com>
@cgillum cgillum requested review from a team as code owners May 22, 2023 20:25
Signed-off-by: Chris Gillum <cgillum@microsoft.com>
Copy link
Member

@msfussell msfussell left a 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

@github-actions
Copy link

Stale PR, paging all reviewers

@github-actions github-actions bot added the stale label May 29, 2023
@msfussell msfussell added this to the 1.11 milestone May 29, 2023
Copy link
Member

@msfussell msfussell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@msfussell msfussell merged commit e19b331 into dapr:v1.11 May 31, 2023
@hhunter-ms hhunter-ms mentioned this pull request Jun 7, 2023
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants