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.

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

Enhancement: aad m365group add visibilty option #5477

Closed
milanholemans opened this issue Sep 11, 2023 · 6 comments
Closed

Enhancement: aad m365group add visibilty option #5477

milanholemans opened this issue Sep 11, 2023 · 6 comments
Assignees
Milestone

Comments

@milanholemans
Copy link
Contributor

Currently, we can create private and public Microsoft 365 groups by specifying or omitting the --isPrivate flag.
However, in fact, you can create 3 types of Microsoft 365 groups, reading the docs:

Property Type Description
visibility String Specifies the group join policy and group content visibility for groups. Possible values are: Private, Public, or HiddenMembership. HiddenMembership can be set only for Microsoft 365 groups, when the groups are created. It can't be updated later. Other values of visibility can be updated after group creation. If visibility value is not specified during group creation on Microsoft Graph, a security group is created as Private by default and Microsoft 365 group is Public. Groups assignable to roles are always Private. To learn more, see group visibility options.

Therefore I suggest that we remove --isPrivate and replace it with:

Option Description
--visibility [visibility] Specifies the group join policy and group content visibility for groups. Possible values are: Private, Public, or HiddenMembership. Defaults to Public.
  • Let's also add a remark that warns the user that you cannot change the group type when you choose HiddenMembership.
  • Let's add a remark that explains all group visibilities
  • Let's recheck all sample scripts for breaking changes.
@milanholemans milanholemans added enhancement needs peer review Needs second pair of eyes to review the spec or PR labels Sep 11, 2023
@milanholemans milanholemans added this to the v7 milestone Sep 11, 2023
@milanholemans
Copy link
Contributor Author

@pnp/cli-for-microsoft-365-maintainers any remarks?

@Jwaegebaert
Copy link
Contributor

Good suggestion! Let's be sure that we clearly explain the effects of the type HiddenMembership

@Jwaegebaert Jwaegebaert added help wanted and removed needs peer review Needs second pair of eyes to review the spec or PR labels Sep 12, 2023
@SmitaNachan
Copy link
Contributor

Can I work on it?

@martinlingstuyl
Copy link
Contributor

Hi @SmitaNachan, just wanted to give you and ask for a timeline on this: v7 is around the corner, we'd like to start a code freeze in about a week to avoid changing things to short before the release.

How are you doing on this PR and the other breaking change? Do let us know!

@SmitaNachan
Copy link
Contributor

Thanks @martinlingstuyl, Raised PR #5497 😊

@martinlingstuyl
Copy link
Contributor

Fantastic!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants