Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Enhance entra m365group user remove by specifying multiple users #6058

Open
1 task
Tracked by #5432
milanholemans opened this issue May 18, 2024 · 1 comment · May be fixed by #6356
Open
1 task
Tracked by #5432

Enhance entra m365group user remove by specifying multiple users #6058

milanholemans opened this issue May 18, 2024 · 1 comment · May be fixed by #6356
Assignees
@MathijsVerbeeck
Labels
enhancement work in progress
Milestone
v10

Comments

@milanholemans
Copy link
Contributor

milanholemans commented May 18, 2024 •
edited by MathijsVerbeeck
Loading

Currently, the command entra m365group user remove only supports specifying one user by UPN. Let's enhance this command so it can accept multiple users by UPN or ID.
In addition, let's also add the possibility to reference the group by display name.

Options to add

Option Description
--groupName [groupName] The display name of the Microsoft 365 group. Specify only one of the following: groupId, groupName, teamId, or teamName.
--teamName [teamName] The display name of the Microsoft Teams team. Specify only one of the following: groupId, groupName, teamId, or teamName.
--ids [ids] Microsoft Entra IDs of users. You can also pass a comma-separated list of IDs. Specify either ids or userNames but not both.
--userNames [userNames] The user principal names of users. You can also pass a comma-separated list of UPNs. Specify either ids or userNames but not both.

Deprecate current option

This will come at a cost, we'll have to deprecate the currently required option -n, --userName <userName>.
Let's make the option optional and deprecate it.
@milanholemans milanholemans added enhancement needs peer review Needs second pair of eyes to review the spec or PR labels May 18, 2024
@milanholemans milanholemans mentioned this issue May 18, 2024
Open
12 tasks
@Jwaegebaert Jwaegebaert added help wanted and removed needs peer review Needs second pair of eyes to review the spec or PR labels May 22, 2024
@MathijsVerbeeck
Copy link
Contributor

Can I work on this one?

MathijsVerbeeck added a commit to MathijsVerbeeck/cli-microsoft365 that referenced this issue Sep 17, 2024
@MathijsVerbeeck
Enhances command entra m365group user remove. Closes pnp#6058
f87c989
@MathijsVerbeeck MathijsVerbeeck linked a pull request Sep 17, 2024 that will close this issue
Open
MathijsVerbeeck added a commit to MathijsVerbeeck/cli-microsoft365 that referenced this issue Sep 17, 2024
@MathijsVerbeeck
Enhances command entra m365group user remove. Closes pnp#6058
55fc4d3
@MathijsVerbeeck MathijsVerbeeck mentioned this issue Sep 17, 2024
Open
MathijsVerbeeck added a commit to MathijsVerbeeck/cli-microsoft365 that referenced this issue Sep 17, 2024
@MathijsVerbeeck
Enhances command entra m365group user remove. Closes pnp#6058
9cc487c
@waldekmastykarz waldekmastykarz added this to the v10 milestone Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@MathijsVerbeeck MathijsVerbeeck

Labels
enhancement work in progress
Projects
None yet
Milestone
v10
Development

Successfully merging a pull request may close this issue.

Enhances command entra m365group user remove MathijsVerbeeck/cli-microsoft365
4 participants
@MathijsVerbeeck @waldekmastykarz @milanholemans @Jwaegebaert