Skip to content

Conversation

@tmaestrini
Copy link
Contributor

Added minimal permissions for several entra app permission commands.

Closes #6924

Copilot AI review requested due to automatic review settings October 21, 2025 22:45
Copy link

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 permissions documentation for three Entra app permission management commands. The documentation follows the project's standard format by including permission tables for both Delegated and Application authentication types.

Key changes:

  • Added permissions sections to entra app permission add, entra app permission list, and entra app permission remove commands
  • Documented Microsoft Graph permissions required for each command

Reviewed Changes

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

File Description
docs/docs/cmd/entra/app/app-permission-add.mdx Added permissions section documenting Application.ReadWrite.All requirement
docs/docs/cmd/entra/app/app-permission-list.mdx Added permissions section documenting Application.Read.All requirement
docs/docs/cmd/entra/app/app-permission-remove.mdx Added permissions section documenting Application.ReadWrite.All requirement

@milanholemans
Copy link
Contributor

Thanks @tmaestrini! We'll try to review it ASAP.
Seems like the docs build is failing. Could you take a look at it? At first sight, it looks like you forgot to import the tabs component in the page.

@milanholemans milanholemans marked this pull request as draft October 21, 2025 23:05
@tmaestrini tmaestrini marked this pull request as ready for review October 22, 2025 06:07
Copilot AI review requested due to automatic review settings October 22, 2025 06:07
Copy link

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

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

@tmaestrini
Copy link
Contributor Author

tmaestrini commented Oct 22, 2025

Thanks, @milanholemans – done and sorry for not having paid attention 🤦🏻‍♂️!

@Adam-it Adam-it self-assigned this Oct 24, 2025
@martinlingstuyl martinlingstuyl added the hacktoberfest-accepted Accept for hacktoberfest, will merge later label Oct 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs hacktoberfest-accepted Accept for hacktoberfest, will merge later

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Minimal permissions for 'entra app permission' commands

4 participants