-
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] Add purge/pause/resume/raise event to docs #3332
Conversation
Signed-off-by: Hannah Hunter <hannahhunter@microsoft.com>
Signed-off-by: Hannah Hunter <hannahhunter@microsoft.com>
Signed-off-by: Hannah Hunter <hannahhunter@microsoft.com>
daprdocs/content/en/developing-applications/building-blocks/workflow/howto-manage-workflow.md
Show resolved
Hide resolved
Signed-off-by: Hannah Hunter <hannahhunter@microsoft.com>
Stale PR, paging all reviewers |
Signed-off-by: Hannah Hunter <hannahhunter@microsoft.com>
Signed-off-by: Hannah Hunter <hannahhunter@microsoft.com>
Signed-off-by: Hannah Hunter <hannahhunter@microsoft.com>
Signed-off-by: Hannah Hunter <hannahhunter@microsoft.com>
Signed-off-by: Hannah Hunter <hannahhunter@microsoft.com>
Signed-off-by: Hannah Hunter <hannahhunter@microsoft.com>
Signed-off-by: Hannah Hunter <hannahhunter@microsoft.com>
daprdocs/content/en/developing-applications/building-blocks/workflow/howto-manage-workflow.md
Outdated
Show resolved
Hide resolved
daprdocs/content/en/developing-applications/building-blocks/workflow/howto-manage-workflow.md
Outdated
Show resolved
Hide resolved
@hhunter-ms it should be // Pause
await daprClient.PauseWorkflowAsync(orderId, workflowComponent); |
daprdocs/content/en/developing-applications/building-blocks/workflow/howto-manage-workflow.md
Show resolved
Hide resolved
Co-authored-by: Chris Gillum <cgillum@gmail.com> Signed-off-by: Mark Fussell <markfussell@gmail.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.
More comments. Still do not understand the Raise an event usage.
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.
More comments. Still do not understand the Raise an event usage.
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.
More comments. Still do not understand the Raise an event usage.
daprdocs/content/en/developing-applications/building-blocks/workflow/howto-manage-workflow.md
Outdated
Show resolved
Hide resolved
Signed-off-by: Hannah Hunter <hannahhunter@microsoft.com>
@msfussell @cgillum @RyanLettieri thanks for the reviews! can you glance over it again to make sure I've addressed all your suggestions? |
LGTM |
daprdocs/content/en/developing-applications/building-blocks/workflow/howto-manage-workflow.md
Outdated
Show resolved
Hide resolved
daprdocs/content/en/developing-applications/building-blocks/workflow/howto-manage-workflow.md
Outdated
Show resolved
Hide resolved
Signed-off-by: Hannah Hunter <hannahhunter@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.
LGTM.
Description
Add purge/pause/resume API methods to documentation:
Need:
Response body examples for Workflow API doc
Issue reference
PR will close: #3316 #3350 #3349