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

resourceProvisioningOptions missing from Group #289

Open
waldekmastykarz opened this issue Jan 14, 2022 · 7 comments
Open

resourceProvisioningOptions missing from Group #289

waldekmastykarz opened this issue Jan 14, 2022 · 7 comments
Assignees

Comments

@waldekmastykarz
Copy link

It seems like the resourceProvisioningOptions property is missing from the Group entity

@ghost ghost added the ToTriage label Jan 14, 2022
@MIchaelMainer
Copy link
Contributor

Global for all libraries.

That property is not set on the v1.0 metadata for Group entity. It is there for beta. Is it supposed to be there for v1.0?

@nikithauc nikithauc self-assigned this Jan 21, 2022
@waldekmastykarz
Copy link
Author

It's included in the v1.0 API reference so I'd expect it to be in the typings too.

@zengin
Copy link
Contributor

zengin commented Jan 28, 2022

It is also returned by the /v1.0/groups API call, so it doesn't look like a case of beta documentation leaking to V1. @nikithauc, I think we should ask the service team to update their metadata.

@nikithauc
Copy link
Contributor

It is also returned by the /v1.0/groups API call, so it doesn't look like a case of beta documentation leaking to V1. @nikithauc, I think we should ask the service team to update their metadata.

Is the missing resource breaking anything ?

@waldekmastykarz
Copy link
Author

It's not breaking anything for me, but since it's missing, I need to cast the response from Graph to any which defies the purpose of using typings.

@zengin
Copy link
Contributor

zengin commented Jan 31, 2022

The effect is similar in other languages, the property will show up not as a direct property of the Group object, but in the generic AdditionalData dictionary, which requires additional steps to wrap and unwrap.

@MIchaelMainer
Copy link
Contributor

I started an email about this. Let me add y'all.

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

4 participants