microsoftgraph / msgraph-sdk-powershell

Powershell SDK for Microsoft Graph
https://www.powershellgallery.com/packages/Microsoft.Graph
Other
702 stars 168 forks source link

v2.0.0: Missing parameters in Cmdlets #1822

Closed NikCharlebois closed 1 year ago

NikCharlebois commented 1 year ago

Several cmdlets are missing parameters in comparison to their 1.22+ alternatives:

peombwa commented 1 year ago

Thanks for bringing this to our attention.

The parameters a missing due to the introduction of readOnly nav properties in https://github.com/microsoftgraph/msgraph-sdk-powershell/issues/1605. This was added to reduce the number of parameters surfaced on update/set/new commands.

@NikCharlebois, were the above parameter accepted by the service when set? According to the API reference:

We will reverse the change made in https://github.com/microsoftgraph/msgraph-sdk-powershell/issues/1605 since some workloads don't fully comply with OData conventions as stated in - https://github.com/microsoft/OpenAPI.NET.OData/issues/254.

peombwa commented 1 year ago

Closing as fixed by https://github.com/microsoftgraph/msgraph-sdk-powershell/releases/tag/2.0.0-preview6.

➜ Get-Help New-MgDirectoryAdministrativeUnit

NAME
    New-MgDirectoryAdministrativeUnit

SYNOPSIS
    Use this API to create a new administrativeUnit.

SYNTAX
    New-MgDirectoryAdministrativeUnit [-AdditionalProperties <Hashtable>] [-DeletedDateTime <DateTime>] [-Description <String>]
    [-DisplayName <String>] [-Extensions <IMicrosoftGraphExtension[]>] [-Id <String>] [-Members <IMicrosoftGraphDirectoryObject[]>]
    [-ScopedRoleMembers <IMicrosoftGraphScopedRoleMembership[]>] [-Visibility <String>] [-Break] [-HttpPipelineAppend <SendAsyncStep[]>]
    [-HttpPipelinePrepend <SendAsyncStep[]>] [-Proxy <Uri>] [-ProxyCredential <PSCredential>] [-ProxyUseDefaultCredentials] [-WhatIf]
    [-Confirm] [<CommonParameters>]

The reference docs will be updated in the next cycle.