Skip to content

Added Minimal permissions for entra roledefinition commands. Closes #6950#7215

Merged
MartinM85 merged 1 commit intopnp:mainfrom
nanddeepn:issue-6950
Apr 18, 2026
Merged

Added Minimal permissions for entra roledefinition commands. Closes #6950#7215
MartinM85 merged 1 commit intopnp:mainfrom
nanddeepn:issue-6950

Conversation

@nanddeepn
Copy link
Copy Markdown
Contributor

Added Minimal permissions for entra roledefinition commands. Closes #6950

Copilot AI review requested due to automatic review settings April 12, 2026 11:18
Copy link
Copy Markdown

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

Adds the required minimal permissions documentation to the m365 entra roledefinition command reference pages, aligning with the project’s documentation standard of placing a Permissions section immediately before Examples.

Changes:

  • Added a new ## Permissions section for each entra roledefinition command doc page.
  • Documented minimal Delegated and Application permissions for Microsoft Graph using Docusaurus <Tabs>/<TabItem>.
  • Added missing MDX imports (Tabs, TabItem) where needed.

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/roledefinition/roledefinition-add.mdx Adds minimal permissions section (Delegated/Application) for roledefinition add.
docs/docs/cmd/entra/roledefinition/roledefinition-get.mdx Adds minimal permissions section (Delegated/Application) for roledefinition get.
docs/docs/cmd/entra/roledefinition/roledefinition-list.mdx Adds minimal permissions section (Delegated/Application) for roledefinition list.
docs/docs/cmd/entra/roledefinition/roledefinition-remove.mdx Adds minimal permissions section and required MDX imports for roledefinition remove.
docs/docs/cmd/entra/roledefinition/roledefinition-set.mdx Adds minimal permissions section and required MDX imports for roledefinition set.

@milanholemans
Copy link
Copy Markdown
Contributor

Thanks! We'll have a look at it soon!

@MartinM85 MartinM85 self-assigned this Apr 15, 2026
Copy link
Copy Markdown
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.

You rock @nanddeepn 🚀
Approved, I will merge it later.

@MartinM85 MartinM85 merged commit c4b10f0 into pnp:main Apr 18, 2026
13 of 14 checks passed
@nanddeepn nanddeepn deleted the issue-6950 branch April 19, 2026 08:07
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.

Minimal permissions for 'entra roledefinition' commands

4 participants