Add MemberObjectId parameter to Remove-PnPAzureADGroupMember and Remove-PnPAzureADGroupOwner cmdlets; update documentation and utility methods for handling directory objects #5158
Conversation
…ve-PnPAzureADGroupOwner cmdlets; update documentation and utility methods for handling directory objects
There was a problem hiding this comment.
Pull Request Overview
This PR adds support for removing Azure AD group members and owners by Object ID, addressing the limitation where only UPN-based removal was possible. This enables removal of directory objects that don't have a UPN, such as nested groups.
Key Changes:
- Added
MemberObjectIdparameter to both cmdlets with a new parameter set "ByObjectId" - Added two new utility methods to handle Object ID-based removal operations
- Updated documentation with new syntax and examples for the ObjectId parameter
Reviewed Changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| src/Commands/Utilities/Microsoft365GroupsUtility.cs | Added RemoveDirectoryMembers and RemoveDirectoryOwners utility methods to handle removal by Object ID |
| src/Commands/AzureAD/RemoveAzureADGroupOwner.cs | Added parameter set support and MemberObjectId parameter for removing owners by Object ID |
| src/Commands/AzureAD/RemoveAzureADGroupMember.cs | Added parameter set support and MemberObjectId parameter for removing members by Object ID |
| documentation/Remove-PnPAzureADGroupOwner.md | Added syntax, examples, and parameter documentation for the new MemberObjectId parameter |
| documentation/Remove-PnPAzureADGroupMember.md | Added syntax, examples, and parameter documentation for the new MemberObjectId parameter |
| ## SYNTAX | ||
|
|
||
| ```powershell | ||
| Remove-PnPAzureADGroupOwner -Identity <AzureADGroupPipeBind> -Users <String[]> [-Verbose] |
There was a problem hiding this comment.
[nitpick] The syntax section should include parameter set names to clarify which parameters belong to which set. Consider adding '### ByUPN' and '### ByObjectId' headings above each syntax example, similar to standard PowerShell documentation format.
|
|
||
| ```powershell | ||
| Remove-PnPAzureADGroupMember -Identity <AzureADGroupPipeBind> -Users <String[]> | ||
| Remove-PnPAzureADGroupMember -Identity <AzureADGroupPipeBind> -Users <String[]> |
There was a problem hiding this comment.
[nitpick] The syntax section should include parameter set names to clarify which parameters belong to which set. Consider adding '### ByUPN' and '### ByObjectId' headings above each syntax example, similar to standard PowerShell documentation format.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Type
Related Issues?
Fixes #5099
What is in this Pull Request ?
Added support to remove users by Object ID