Skip to content

Update JSON structure in management group API documentation#128623

Open
meghanagupta95 wants to merge 2 commits into
MicrosoftDocs:mainfrom
meghanagupta95:patch-1
Open

Update JSON structure in management group API documentation#128623
meghanagupta95 wants to merge 2 commits into
MicrosoftDocs:mainfrom
meghanagupta95:patch-1

Conversation

@meghanagupta95

Copy link
Copy Markdown
Contributor

The PUT MG's request body was missing the "details" field. This would have led to the MG being created under the root MG by default. Placing the "parent" JSON under the "details" key ensures the provided parent id is respected.

The PUT MG's request body was missing the "details" field. This would have led to the MG being created under the root MG by default. Placing the "parent" JSON under the "details" key ensures the provided parent id is respected.
@prmerger-automator

Copy link
Copy Markdown
Contributor

@meghanagupta95 : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change.

@prmerger-automator

Copy link
Copy Markdown
Contributor

@meghanagupta95 : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change.

@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit ccc7aee:

✅ Validation status: passed

File Status Preview URL Details
articles/governance/management-groups/create-management-group-rest-api.md ✅Succeeded

For more details, please refer to the build report.

@Court72 Court72 requested a review from Copilot July 9, 2026 14:36

Copilot AI 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.

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Updates the Management Group PUT request body example to correctly nest parent under properties.details, preventing unintended creation under the root management group.

Changes:

  • Add properties.details wrapper to the request body JSON example
  • Ensure the JSON example is structurally complete (adds the missing closing brace)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Court72

Court72 commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

@iamwilliew

Can you review the proposed changes?

Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@prmerger-automator prmerger-automator Bot added the aq-pr-triaged tracking label for the PR review team label Jul 9, 2026
@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 9990b05:

✅ Validation status: passed

File Status Preview URL Details
articles/governance/management-groups/create-management-group-rest-api.md ✅Succeeded

For more details, please refer to the build report.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants