Skip to content

Added minimal permissions for 'spo file sharinglink' commands. Closes #7271 - #7457

Open
nanddeepn wants to merge 1 commit into
pnp:mainfrom
nanddeepn:issue-7271
Open

Added minimal permissions for 'spo file sharinglink' commands. Closes #7271#7457
nanddeepn wants to merge 1 commit into
pnp:mainfrom
nanddeepn:issue-7271

Conversation

@nanddeepn

Copy link
Copy Markdown
Contributor

Added minimal permissions for spo file sharinglink commands. Closes #7271

Copilot AI review requested due to automatic review settings July 20, 2026 11:52

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 a new Permissions section to the spo file sharinglink command documentation pages to document the minimal delegated and application permissions required, aligning with issue #7271’s guidance (place Permissions right before Examples).

Changes:

  • Added a Permissions section (with Delegated/Application tabs) to the six spo file sharinglink command reference pages.
  • Added missing @theme/Tabs / @theme/TabItem imports where needed for the new tabbed Permissions content.

Reviewed changes

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

Show a summary per file
File Description
docs/docs/cmd/spo/file/file-sharinglink-add.mdx Adds Permissions section for spo file sharinglink add.
docs/docs/cmd/spo/file/file-sharinglink-clear.mdx Adds Permissions section (and Tabs imports) for spo file sharinglink clear.
docs/docs/cmd/spo/file/file-sharinglink-get.mdx Adds Permissions section for spo file sharinglink get.
docs/docs/cmd/spo/file/file-sharinglink-list.mdx Adds Permissions section for spo file sharinglink list.
docs/docs/cmd/spo/file/file-sharinglink-remove.mdx Adds Permissions section (and Tabs imports) for spo file sharinglink remove.
docs/docs/cmd/spo/file/file-sharinglink-set.mdx Adds Permissions section for spo file sharinglink set.

Comment on lines +44 to +53
| Resource | Permissions |
|------------|-----------------|
| SharePoint | Files.ReadWrite |

</TabItem>
<TabItem value="Application">

| Resource | Permissions |
|------------|---------------------|
| SharePoint | Files.ReadWrite.All |
Comment on lines +41 to +50
| Resource | Permissions |
|------------|-----------------|
| SharePoint | Files.ReadWrite |

</TabItem>
<TabItem value="Application">

| Resource | Permissions |
|------------|---------------------|
| SharePoint | Files.ReadWrite.All |
Comment on lines +38 to +47
| Resource | Permissions |
|------------|-------------|
| SharePoint | Files.Read |

</TabItem>
<TabItem value="Application">

| Resource | Permissions |
|------------|----------------|
| SharePoint | Files.Read.All |
Comment on lines +38 to +47
| Resource | Permissions |
|------------|-------------|
| SharePoint | Files.Read |

</TabItem>
<TabItem value="Application">

| Resource | Permissions |
|------------|----------------|
| SharePoint | Files.Read.All |
Comment on lines +41 to +50
| Resource | Permissions |
|------------|-----------------|
| SharePoint | Files.ReadWrite |

</TabItem>
<TabItem value="Application">

| Resource | Permissions |
|------------|---------------------|
| SharePoint | Files.ReadWrite.All |
Comment on lines +41 to +50
| Resource | Permissions |
|------------|-----------------|
| SharePoint | Files.ReadWrite |

</TabItem>
<TabItem value="Application">

| Resource | Permissions |
|------------|---------------------|
| SharePoint | Files.ReadWrite.All |
@milanholemans

Copy link
Copy Markdown
Contributor

Thanks, we'll review it soon!

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 'spo file sharinglink' commands

3 participants