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

The new 'uxSetting' object may not be fetched through 'entra' which is marked as deprecated #1075

Closed
cnotin opened this issue Nov 4, 2024 · 4 comments
Labels
dependency:metadata Awaiting fix from core dependency project module status:waiting-for-dependency An issue that has core project dependency that's currently blocking it type:bug A broken experience

Comments

@cnotin
Copy link

cnotin commented Nov 4, 2024

Describe the bug

I want to access the uxSetting resource which was recently added to the API: https://developer.microsoft.com/en-us/graph/changelog/?search=d52616ae-d7cc-4833-bf29-0150cf92f7d7
Thankfully I noticed it's already available in the SDK since September: https://github.com/microsoftgraph/msgraph-beta-sdk-java/blob/main/src/main/java/com/microsoft/graph/beta/generated/models/UxSetting.java
I found I need to call the UxSettingRequestBuilder, only exposed via the EntraRequestBuilder however entra().uxSetting() is marked as deprecated potentially for the reason "PrivatePreview":

public class EntraRequestBuilder extends BaseRequestBuilder {
/**
* Provides operations to manage the uxSetting property of the microsoft.graph.entra entity.
* @return a {@link UxSettingRequestBuilder}
* @deprecated
* as of 2023-11/PrivatePreview:BulkJobs
*/
@Deprecated
@jakarta.annotation.Nonnull
public UxSettingRequestBuilder uxSetting() {

Perhaps you need to refresh something? Or the Graph API team must remove this private preview tag?

Expected behavior

I should be able to call this:

graphServiceClient.admin().entra().uxSetting()

without a deprecated warning in my IDE

How to reproduce

  1. Get the latest version of the SDK
  2. Try to get the UxSetting:
graphServiceClient.admin().entra().uxSetting()
  1. observe your IDE warning
    image

SDK Version

6.21.0

Latest version known to work for scenario above?

No response

Known Workarounds

ignore the warning

Debug output

Click to expand log ```
</details>


### Configuration

_No response_

### Other information

_No response_
@cnotin cnotin added the type:bug A broken experience label Nov 4, 2024
@baywet
Copy link
Member

baywet commented Nov 10, 2024

Hi @cnotin
Thank you for using the SDK and for reaching out.

Created an issue on the service team, they forgot to remove the metadata annotation which leads to the deprecated annotation when releasing.
https://portal.microsofticm.com/imp/v5/incidents/details/563621519/summary

@baywet baywet added status:waiting-for-dependency An issue that has core project dependency that's currently blocking it dependency:metadata Awaiting fix from core dependency project module labels Nov 10, 2024
@baywet
Copy link
Member

baywet commented Nov 13, 2024

Short update: the service team has started putting the changes together, we're now waiting on approval and deployment. This might take a while with the upcoming holidays.

@cnotin
Copy link
Author

cnotin commented Nov 13, 2024

Good news! Thanks Vincent

@baywet
Copy link
Member

baywet commented Jan 6, 2025

Closing as corrected with 3129de5

@baywet baywet closed this as completed Jan 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependency:metadata Awaiting fix from core dependency project module status:waiting-for-dependency An issue that has core project dependency that's currently blocking it type:bug A broken experience
Projects
None yet
Development

No branches or pull requests

2 participants