Skip to content

Conversation

nanddeepn
Copy link
Contributor

Added minimal permission for entra group commands. Closes #6935

@Copilot Copilot AI review requested due to automatic review settings October 11, 2025 14:45
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds minimal permission requirements documentation to the entra group command reference pages. The changes provide clear guidance on the Microsoft Graph permissions needed for both delegated and application authentication scenarios.

  • Added standardized permissions tables to five entra group command documentation files
  • Specified Group.Read.All for read operations (get, list) and Group.ReadWrite.All for write operations (add, set, remove)
  • Implemented consistent formatting using Docusaurus tabs for delegated vs application permissions

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
docs/docs/cmd/entra/group/group-add.mdx Added permissions section requiring Group.ReadWrite.All for creating groups
docs/docs/cmd/entra/group/group-get.mdx Added permissions section requiring Group.Read.All for retrieving group information
docs/docs/cmd/entra/group/group-list.mdx Added permissions section requiring Group.Read.All for listing groups
docs/docs/cmd/entra/group/group-remove.mdx Added permissions section requiring Group.ReadWrite.All for deleting groups
docs/docs/cmd/entra/group/group-set.mdx Added permissions section requiring Group.ReadWrite.All for updating groups

@MartinM85 MartinM85 self-assigned this Oct 12, 2025
Copy link
Contributor

@MartinM85 MartinM85 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the contribution @nanddeepn 🚀
Only a few minor notes, great job otherwise.

@MartinM85 MartinM85 marked this pull request as draft October 12, 2025 07:44
@nanddeepn nanddeepn marked this pull request as ready for review October 13, 2025 17:46
Copy link
Contributor

@MartinM85 MartinM85 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work. Approved.
I will merge it later.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Minimal permissions for 'entra group' commands

3 participants