Skip to content

Membership management changes #981

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

ShreyasSar26
Copy link
Contributor

New cmdlets to manage membership of containers

Copy link
Contributor

Learn Build status updates of commit dedc559:

✅ Validation status: passed

File Status Preview URL Details
sharepoint/sharepoint-ps/Microsoft.Online.SharePoint.PowerShell/Add-SPOContainerUser ✅Succeeded
sharepoint/sharepoint-ps/Microsoft.Online.SharePoint.PowerShell/Remove-SPOContainerUser ✅Succeeded
sharepoint/sharepoint-ps/Microsoft.Online.SharePoint.PowerShell/Set-SPOContainerUser ✅Succeeded

For more details, please refer to the build report.


## SYNOPSIS

Assigns a particular membership of a container to users.
Copy link
Contributor

@samkabue samkabue Aug 11, 2025

Choose a reason for hiding this comment

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

this file and the others: please use M365 Copilot to polish the strings. Here's example for this one:

prompt:

I have this cmdlet:

Add-SPOContainerUser [-ContainerId] -LoginName -Role []
here's the current description:

"Assigns a particular membership of a container to users."

Rewrite this description based on Microsoft 365 Style Guide.

output:

Here’s a revised description for the Add-SPOContainerUser cmdlet, aligned with the https://learn.microsoft.com/style-guide/welcome/:

Adds a user to a container with a specified role.

This version uses active voice, avoids unnecessary complexity, and clearly states what the cmdlet does. Let me know if you'd like help updating related cmdlet descriptions or examples.

Copy link
Contributor

Choose a reason for hiding this comment

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

other suggestions from M365 Copilot:

ContainerId: The unique identifier of the container to which the user is being added.
LoginName: The user’s login name to assign to the container.
Role: The role to assign to the user within the container. Valid roles are:

  • Owner: Has full control over the container and its contents.
  • Manager: Can add, update, and delete content, and manage permissions, but can't delete the container.
  • Writer: Can add, update, and delete content in the container.
  • Reader: Can only view content in the container.

@samkabue
Copy link
Contributor

samkabue commented Aug 11, 2025

Also update the Table of Contents file -- Microsoft.Online.SharePoint.PowerShell.md

@samkabue samkabue added Back to submitter The pull request didn't pass the review criteria and has been returned to the writer for updates Microsoft submitter labels Aug 11, 2025
Copy link
Contributor

PoliCheck Scan Report

The following reports lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 issues. Other issues are also a high priority. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a comment that you post to the PR. The feedback helps reduce false positive instances in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

Copy link
Contributor

Learn Build status updates of commit 1cac876:

✅ Validation status: passed

File Status Preview URL Details
sharepoint/sharepoint-ps/Microsoft.Online.SharePoint.PowerShell/Add-SPOContainerUser ✅Succeeded
sharepoint/sharepoint-ps/Microsoft.Online.SharePoint.PowerShell/Remove-SPOContainerUser ✅Succeeded
sharepoint/sharepoint-ps/Microsoft.Online.SharePoint.PowerShell/Set-SPOContainerUser ✅Succeeded

For more details, please refer to the build report.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Back to submitter The pull request didn't pass the review criteria and has been returned to the writer for updates Microsoft submitter
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants