Skip to content

Added minimal permissions for 'spo group' commands. Closes #7277#7447

Open
nanddeepn wants to merge 2 commits into
pnp:mainfrom
nanddeepn:issue-7277
Open

Added minimal permissions for 'spo group' commands. Closes #7277#7447
nanddeepn wants to merge 2 commits into
pnp:mainfrom
nanddeepn:issue-7277

Conversation

@nanddeepn

Copy link
Copy Markdown
Contributor

Added minimal permissions for spo group commands. Closes #7277

Copilot AI review requested due to automatic review settings July 13, 2026 06:22

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 updates the SharePoint Online spo group command documentation to include the new Permissions section that documents minimal delegated and application permissions required to run each command, aligning with the documentation standard introduced in #7277.

Changes:

  • Added a Permissions section (positioned immediately before Examples) to all spo group command docs in scope.
  • Documented minimal delegated vs. application permissions using Docusaurus <Tabs>/<TabItem> for consistency.
  • Cleaned up example headings formatting in group-add.mdx (removed accidental list/table formatting and added periods).

Reviewed changes

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

Show a summary per file
File Description
docs/docs/cmd/spo/group/group-add.mdx Adds minimal delegated/app permissions for creating a SharePoint group; fixes example heading formatting.
docs/docs/cmd/spo/group/group-get.mdx Adds minimal delegated/app permissions for retrieving a SharePoint group.
docs/docs/cmd/spo/group/group-list.mdx Adds minimal delegated/app permissions for listing SharePoint groups.
docs/docs/cmd/spo/group/group-remove.mdx Adds minimal delegated/app permissions for removing a SharePoint group (and imports Tabs components).
docs/docs/cmd/spo/group/group-set.mdx Adds minimal delegated/app permissions for updating a SharePoint group (and imports Tabs components).
docs/docs/cmd/spo/group/group-member-add.mdx Adds minimal delegated/app permissions for adding members to a SharePoint group.
docs/docs/cmd/spo/group/group-member-list.mdx Adds minimal delegated/app permissions for listing group members.
docs/docs/cmd/spo/group/group-member-remove.mdx Adds minimal delegated/app permissions for removing members from a SharePoint group (and imports Tabs components).

@milanholemans

Copy link
Copy Markdown
Contributor

Thanks, we'll review it soon!

@Adam-it Adam-it self-assigned this Jul 15, 2026

@Adam-it Adam-it left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@nanddeepn Awesome work so far but I am wondering about the AllSites.Manage permissions scopes. I think it only allows to read and write items and lists in all site collections. But groups and especially managing group members is a permission kind of thing and I would expect AllSites.FullContro would be required. Could we give this a double check?

@Adam-it
Adam-it marked this pull request as draft July 19, 2026 20:35
@nanddeepn

Copy link
Copy Markdown
Contributor Author

Hi @Adam-it
Thank you for the correction. Updated the permissions.

@nanddeepn
nanddeepn marked this pull request as ready for review July 20, 2026 11:26
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 group' commands

4 participants