Skip to content

Conversation

@nanddeepn
Copy link
Contributor

Added minimal permission for graph openextension commands. Closes #6980

Copilot AI review requested due to automatic review settings October 15, 2025 10:41
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 permission documentation for the graph openextension commands. The changes address issue #6980 by documenting the minimal Microsoft Graph permissions required for each command.

Key changes:

  • Added permission tables for read operations (get, list) requiring User.Read, Group.Read.All, and Directory.Read.All
  • Added permission tables for write operations (add, set, remove) requiring User.ReadWrite, Group.ReadWrite.All, and Directory.AccessAsUser.All
  • Documented both delegated and application permissions for all five commands

Reviewed Changes

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

Show a summary per file
File Description
openextension-add.mdx Added permissions section for creating open extensions
openextension-get.mdx Added permissions section for retrieving a single open extension
openextension-list.mdx Added permissions section for listing open extensions
openextension-remove.mdx Added permissions section for removing open extensions
openextension-set.mdx Added permissions section for updating open extensions

@Adam-it
Copy link
Member

Adam-it commented Oct 19, 2025

Thank you for opening the PR 👏👏👏
You Rock 🤩
We will review it ASAP 🚀

Copilot AI review requested due to automatic review settings October 20, 2025 10:52
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 5 out of 5 changed files in this pull request and generated no new comments.

@Jwaegebaert Jwaegebaert self-assigned this Oct 23, 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 'graph openextension' commands

5 participants