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

New command: m365 entra multitenant get #6007

Open
MartinM85 opened this issue Apr 30, 2024 · 5 comments · May be fixed by #6029
Open

New command: m365 entra multitenant get #6007

MartinM85 opened this issue Apr 30, 2024 · 5 comments · May be fixed by #6029

Comments

@MartinM85
Copy link
Contributor

MartinM85 commented Apr 30, 2024

Usage

m365 entra multitenant get [options]

Description

Gets properties of the multitenant organization

Options

no command specific options

Examples

Gets properties of the multitenant organization

m365 entra multitenant get

Default properties

Additional Info

Multitenant organization is a brand new feature in Microsoft Entra ID

https://learn.microsoft.com/en-us/entra/identity/multi-tenant-organizations/overview

https://learn.microsoft.com/en-us/graph/api/multitenantorganization-get?view=graph-rest-1.0

The signed-in user must be assigned at least the Security Administrator role. It requires new permission MultiTenantOrganization.ReadBasic.All

I will work on it

@waldekmastykarz
Copy link
Member

Thank you for the suggestion, @MartinM85! Let's clarify the name using the thread in #6006 and we'll take it from there

@MartinM85 MartinM85 changed the title New command: m365 multitenant get New command: m365 entra multitenant get May 3, 2024
@Adam-it Adam-it added needs peer review Needs second pair of eyes to review the spec or PR and removed on hold labels May 4, 2024
@Adam-it
Copy link
Contributor

Adam-it commented May 4, 2024

the naming is set so let's start reviewing the spec 👍

@Jwaegebaert
Copy link
Contributor

One small detail, as this is a get command we won't need to define any default properties. But besides that, it looks good to go.

@Jwaegebaert Jwaegebaert added good first issue work in progress and removed needs peer review Needs second pair of eyes to review the spec or PR labels May 6, 2024
@Jwaegebaert
Copy link
Contributor

All yours @MartinM85!

@Jwaegebaert
Copy link
Contributor

Another detail, maybe change the description to something like Gets properties of the multitenant organization.

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

Successfully merging a pull request may close this issue.

4 participants