Skip to content

Fixed an issue in JsonPolicyWriter.WritePolicyToString() method where it was not using prettyPrint parameter for indentation. #3831

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
May 21, 2025
Merged

Fixed an issue in JsonPolicyWriter.WritePolicyToString() method where it was not using prettyPrint parameter for indentation. #3831

merged 1 commit into from
May 21, 2025
+46 −1

Conversation

ashishdhingra
Copy link
Contributor

@ashishdhingra ashishdhingra commented May 20, 2025 •
edited
Loading

Description

Fixed an issue in JsonPolicyWriter.WritePolicyToString() method where it was not using prettyPrint parameter for indentation.

Motivation and Context

Issue #3805.

Testing

  • Added unit tests.
  • Dry-run DRY_RUN-4f6f7952-cc98-4d78-922a-44d23aaf629c completed successfully.

Screenshots (if appropriate)

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • My code follows the code style of this project
  • My change requires a change to the documentation
  • I have updated the documentation accordingly
  • I have read the README document
  • I have added tests to cover my changes
  • All new and existing tests passed

License

  • I confirm that this pull request can be released under the Apache 2 license

@ashishdhingra
Fixed an issue in JsonPolicyWriter.WritePolicyToString() method where…
e8b80e1 
… it was not using prettyPrint parameter for indentation.
@ashishdhingra ashishdhingra merged commit c822aec into development May 21, 2025
3 checks passed
@ashishdhingra ashishdhingra deleted the user/ashdhin/V4-DOTNET-8132-JsonPolicyWriter-PrettyPrint branch May 21, 2025 16:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@normj normj normj approved these changes

@afroz429 afroz429 afroz429 approved these changes

Assignees
No one assigned
Labels
v4
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ashishdhingra @normj @afroz429