Skip to content

Fixes removal of boolean and empty Json object values from request body #3224

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

Merged
merged 5 commits into from
Mar 18, 2025

Conversation

timayabi2020
Copy link
Contributor

@timayabi2020 timayabi2020 commented Mar 15, 2025

Fixes #3201, #3212 #3215, #3202, #2902

The PR also updates tests to cover both boolean and empty Json object valued property scenarios.

Ensuring boolean property values are not dropped

image

Ensures empty Json object/list as a property value is not dropped

Example 1:
image

Example 2:
image

@timayabi2020 timayabi2020 changed the title Fixes removal of boolean values from request body Fixes removal of boolean and empty Json object values from request body Mar 15, 2025
@timayabi2020 timayabi2020 marked this pull request as ready for review March 15, 2025 14:16
@timayabi2020 timayabi2020 requested a review from a team as a code owner March 15, 2025 14:16
@timtroch
Copy link

This doesn't seem to be resolved yet:

image

@Xuzhou-Huang
Copy link

Will this fix get released in next version?

@zaaj
Copy link

zaaj commented Mar 18, 2025

@timtroch ,

Looks like it takes more than an hour for Microsoft to go from accepting a bug-fix pull-request into "dev" (a common name for a development version of the software) to releasing a new production version of the Microsoft.Graph powershell module into the Powershell Gallery.

@Xuzhou-Huang, I'm not a Microsoft employee, but since the pull request got accepted a few hours ago, I would expect this bug fix will make it's way into a future release - I don't know their release cadence so can't say when or in what version these fixes will be included.

In the mean time, uninstalling the 2.26.1 release and re-installing 2.25.0 should help.

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.

Set-MgBetaUserLicense not working
5 participants