Skip to content

fix(workflow): add missing FailureDetails property to proto converter#1831

Merged
WhitWaldo merged 2 commits into
dapr:masterfrom
Eckii24:fix/workflow-proto-converter-failure-details-property
May 27, 2026
Merged

fix(workflow): add missing FailureDetails property to proto converter#1831
WhitWaldo merged 2 commits into
dapr:masterfrom
Eckii24:fix/workflow-proto-converter-failure-details-property

Conversation

@Eckii24

@Eckii24 Eckii24 commented May 26, 2026

Copy link
Copy Markdown
Contributor

Description

Please explain the changes you've made

Adds the missing mapping of FailureDetails in ProtoConverters.

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: #1830

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

Signed-off-by: Matthias Eck <matthias.eck@zeiss.com>
@Eckii24
Eckii24 requested review from a team as code owners May 26, 2026 10:34
@WhitWaldo WhitWaldo added this to the v1.18 milestone May 26, 2026

@WhitWaldo WhitWaldo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good to me - thank you for taking the time to file the issue and put this PR together.

@WhitWaldo
WhitWaldo merged commit 29eb639 into dapr:master May 27, 2026
660 of 669 checks passed
@Eckii24
Eckii24 deleted the fix/workflow-proto-converter-failure-details-property branch May 27, 2026 17:24
@marcduiker

Copy link
Copy Markdown
Contributor

@holopin-bot @Eckii24 sdk-badge Thank you! Here's a digital badge as a small token of appreciation.

@holopin-bot

holopin-bot Bot commented Jun 12, 2026

Copy link
Copy Markdown

Congratulations @Eckii24, the maintainer of this repository has issued you a badge! Here it is: https://holopin.io/claim/cmqbf5zof0006jy0441n740oc

This badge can only be claimed by you, so make sure that your GitHub account is linked to your Holopin account. You can manage those preferences here: https://holopin.io/account.
Or if you're new to Holopin, you can simply sign up with GitHub, which will do the trick!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

WorkflowState.FailureDetails are not populated

4 participants