msgraph-beta-sdk-dotnet icon indicating copy to clipboard operation
msgraph-beta-sdk-dotnet copied to clipboard

IdentityGovernance.EntitlementManagement.AccessPackageCatalogs.GetAsync() marked as obsolete

Open tjrobinson opened this issue 1 year ago • 2 comments

I'm using Microsoft.Graph.Beta 5.49.0-preview, the latest at the time of writing.

This is a snippet of my code:

var response = await _graphClient.IdentityGovernance.EntitlementManagement.AccessPackageCatalogs.GetAsync(
    requestConfiguration =>
    {
        requestConfiguration.QueryParameters.Expand = new[] { "AccessPackageResources" };
    });

I'm getting a warning about IdentityGovernance.EntitlementManagement.AccessPackageCatalogs.GetAsync() being obsolete:

Warning CS0618 : 'AccessPackageCatalogsRequestBuilder.GetAsync(Action<AccessPackageCatalogsRequestBuilder.AccessPackageCatalogsRequestBuilderGetRequestConfiguration>?, CancellationToken)' is obsolete: ' as of 2022-10/PrivatePreview:MicrosofEntitlementManagementCustomextensions on 2023-03-01 and will be removed 2023-12-31'

I've not been able to find a way to resolve this - what should I be calling instead?

Thanks!

tjrobinson avatar Oct 02 '23 10:10 tjrobinson

Thanks for raising this @tjrobinson,

For context, the obsolete annotations on the SDK are pulled from the API metadata available at https://graph.microsoft.com/beta/$metadata. The API changelog does not seem to mention the AccessPackageCatalog type being deprecated though. https://developer.microsoft.com/en-us/graph/changelog/?search=&filterBy=Deprecation,beta,Identity%20and%20access

This would suggest that the deprecated annotation may have been incorrectly applied in the metadata. This will need to be followed up by the API team for clarification.

andrueastman avatar Oct 12 '23 11:10 andrueastman

Hi @andrueastman,

Are there any updates on this? I'm still getting the same warning and have updated to the latest Microsoft.Graph.Beta 5.64.0-preview.

I'm also seeing it on AccessPackageAssignmentRequests and I suspect other types are affected too.

Thanks!

tjrobinson avatar Feb 20 '24 13:02 tjrobinson