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 spe containertype list #5989

Open
Adam-it opened this issue Apr 23, 2024 · 6 comments
Open

New command: m365 spe containertype list #5989

Adam-it opened this issue Apr 23, 2024 · 6 comments

Comments

@Adam-it
Copy link
Contributor

Adam-it commented Apr 23, 2024

Usage

m365 spe containertype list [options]

Description

Lists Container Types

Options

I do not think we will have any option here. The general idea of this command is to list all container types. If needed we may always use the standard --query option to filter the result for example by applicationID

Examples

Lists all Container Types

m365 spe containertype list

Default properties

  • ContainerTypeId
  • ContainerTypeName
  • OwningApplicationId

Additional Info

We may use CSOM for this command passing the following

`${adminUrl}/_vti_bin/client.svc/ProcessQuery`
<Request AddExpandoFieldTypeSuffix="true" SchemaVersion="15.0.0.0" LibraryVersion="16.0.0.0" ApplicationName="SharePoint Online PowerShell (16.0.24322.0)"
    xmlns="http://schemas.microsoft.com/sharepoint/clientquery/2009">
    <Actions>
        <ObjectPath Id="46" ObjectPathId="45" />
        <Method Name="GetSPOContainerTypes" Id="47" ObjectPathId="45">
            <Parameters>
                <Parameter Type="Enum">1</Parameter>
            </Parameters>
        </Method>
    </Actions>
    <ObjectPaths>
        <Constructor Id="45" TypeId="{268004ae-ef6b-4e9b-8425-127220d84719}" />
    </ObjectPaths>
</Request>
@Adam-it Adam-it added new feature needs peer review Needs second pair of eyes to review the spec or PR labels Apr 23, 2024
@Adam-it
Copy link
Contributor Author

Adam-it commented Apr 23, 2024

@andrewconnell wanna take a look at 👆

@andrewconnell
Copy link
Contributor

¯_(ツ)_/¯

Nope... no insights on this one.

@milanholemans
Copy link
Contributor

Aren't these quite a lot of default properties @Adam-it? Can we fit that much info on a default-sized terminal screen?

@Adam-it
Copy link
Contributor Author

Adam-it commented Apr 25, 2024

Aren't these quite a lot of default properties @Adam-it? Can we fit that much info on a default-sized terminal screen?

good comment. I reduced to fewer props.
@pnp/cli-for-microsoft-365-maintainers any other comment before I open this up?

@waldekmastykarz
Copy link
Member

Looking at the default properties, I suggest we drop classification, and use the first three properties (id, name and owning app id). Let's do this 💪

@Adam-it Adam-it added help wanted good first issue and removed needs peer review Needs second pair of eyes to review the spec or PR labels Apr 26, 2024
@reshmee011
Copy link
Contributor

reshmee011 commented May 9, 2024

Since I will be working on the m365 spe containertype get , it makes senses to work on the m365 spe containertype list first. Assign to me.

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

No branches or pull requests

5 participants