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

feat: add JSON serialization configuration and change SerializerSettings visibility to public #1331

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

egyware
Copy link

@egyware egyware commented Jan 22, 2025

  • Added the ConfigureJsonSettings extension to customize JSON serialization settings in WorkflowOptions.
  • Changed SerializerSettings visibility from private to public to allow external access to Newtonsoft.Json configuration.

Reason:
When serializing objects like Outcome or Steps, some could not be serialized correctly. This change enables adjusting Newtonsoft.Json settings to meet specific requirements.

Describe the change
A clear and concise description of what the change is. Any PR submitted without a description of the change will not be reviewed.

Describe your implementation or design
How did you go about implementing the change?

Tests
Did you cover your changes with tests?

Breaking change
Do you changes break compatibility with previous versions?

Additional context
Any additional information you'd like to provide?

…ngs visibility to public

- Added the `ConfigureJsonSettings` extension to customize JSON serialization settings in `WorkflowOptions`.
- Changed `SerializerSettings` visibility from private to public to allow external access to Newtonsoft.Json configuration.

Reason:
When serializing objects like `Outcome` or `Steps`, some could not be serialized correctly. This change enables adjusting Newtonsoft.Json settings to meet specific requirements.
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.

1 participant