docs: add minimal permissions for entra organization list command #6961
Conversation
- Added Permissions section to organization-list.mdx - Documents Organization.Read.All and User.Read permissions for both delegated and application auth - Closes pnp#6947
|
Hi @URAYUSHJAIN , the issue specified 2 commands, but this PR contains only 1 command. Could you add the |
The organization-set command already has the Permissions section properly documented in the main branch with Organization.ReadWrite.All for both delegated and application permissions. Both commands in issue pnp#6947 are now covered: - organization-list: Added permissions (new) - organization-set: Already had permissions (confirmed)
|
@Jwaegebaert sir Good catch! I've reviewed both commands: ✅ organization-list: Added permissions in this PR Added a commit to confirm I've reviewed both. Both commands in #6947 are now covered! |
There was a problem hiding this comment.
Great start @URAYUSHJAIN! I've left a few remarks you can take a look at.
|
|
||
| | Resource | Permissions | | ||
| |-----------------|--------------------------------------------------| | ||
| | Microsoft Graph | Organization.Read.All, User.Read | |
There was a problem hiding this comment.
Application context can't contain "user.read" permissions
| | Microsoft Graph | Organization.Read.All, User.Read | | |
| | Microsoft Graph | Organization.Read.All | |
|
@Jwaegebaert plz merge |
|
First, take a look at the remarks I've left before we can merge these changes @URAYUSHJAIN |
|
|
||
| :::info | ||
|
|
There was a problem hiding this comment.
@URAYUSHJAIN, One more note here. Let's move this admonition below the permissions tab.
Uh oh!
There was an error while loading. Please reload this page.