Skip to content

.Net Processes - Fix Serialization of Input Data - #9614

Merged
Chris (crickman) merged 2 commits into
mainfrom
process-data-serialization-ptdeux
Nov 7, 2024
Merged

Chris (crickman) merged 2 commits into
mainfrom
process-data-serialization-ptdeux

Conversation

@crickman

Copy link
Copy Markdown
Contributor

Motivation and Context

Updated ability to serialize complex types here: #9525

...but missed the remoting boundary for invoke the process. This change addresses this final gap without altering external patterns.

Description

Convert event to JSON when invoking ProcessActor from DaprKernelProcessContext

Contribution Checklist

@crickman Chris (crickman) added PR: ready for review All feedback addressed, ready for reviews .NET Issue or Pull requests regarding .NET code experimental Associated with an experimental feature enhancement processes labels Nov 7, 2024
@crickman Chris (crickman) self-assigned this Nov 7, 2024
@crickman
Chris (crickman) requested a review from a team as a code owner November 7, 2024 19:11
@crickman
Chris (crickman) added this pull request to the merge queue Nov 7, 2024
Merged via the queue into main with commit 1cb0922 Nov 7, 2024
@crickman
Chris (crickman) deleted the process-data-serialization-ptdeux branch November 7, 2024 19:51

This branch was previously deployed

1 inactive deployment
integration — 8e6d255f Deployed Nov 7, 2024 by crickman
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement experimental Associated with an experimental feature .NET Issue or Pull requests regarding .NET code PR: ready for review All feedback addressed, ready for reviews processes

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants