Skip to content

DevEx: Remove need to wrap GetWorkflowStateAsync for non-existent state#1773

Merged
WhitWaldo merged 2 commits intodapr:masterfrom
WhitWaldo:try-get-state
Apr 8, 2026
Merged

DevEx: Remove need to wrap GetWorkflowStateAsync for non-existent state#1773
WhitWaldo merged 2 commits intodapr:masterfrom
WhitWaldo:try-get-state

Conversation

@WhitWaldo
Copy link
Copy Markdown
Contributor

@WhitWaldo WhitWaldo commented Apr 8, 2026

Description

Wrapping RpcException thrown when the SDK fails to retrieve the workflow state to improve DevEx.

Thanks to @olitomlinson for the suggestion!

Issue reference

We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.

Please reference the issue this PR will close: #[issue number]

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • Code compiles correctly
  • Created/updated tests
  • Extended the documentation

…low state to improve DevEx

Signed-off-by: Whit Waldo <whit.waldo@innovian.net>
@WhitWaldo WhitWaldo self-assigned this Apr 8, 2026
@WhitWaldo WhitWaldo requested review from a team as code owners April 8, 2026 00:24
Signed-off-by: Whit Waldo <whit.waldo@innovian.net>
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.

1 participant