-
Notifications
You must be signed in to change notification settings - Fork 468
Update grantedTo structure in permissions overview #9772
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
base: main
Are you sure you want to change the base?
Conversation
Payload for POST request was incorrect; updated to reflect the correct payload, including grantedToIdentities/application with id AND displayName
|
Learn Build status updates of commit 8d00319: ✅ Validation status: passed
For more details, please refer to the build report. |
FaithOmbongi
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @dylanstetts for propsing these fixes. While your changes are valid, I also note that the grantedToIdentities property is deprecated in favor of grantedToIdentitiesV2. If you don't mind, can you update the sample to use grantedToIdentitiesV2? Or if you prefer, I can push this to the product team to update the article. Your contribution is very welcome.
|
Hi @FaithOmbongi, the request payload only accepts grantedToIdentities, so the proposed changes I made are the only supported, working payload to send to the endpoint as of today. I.e.,
|
|
Hi @dylanstetts - I see the property's return type, sharePointIdentitySet, is a little different so likely, application isn't the only required property. I'm adding the SME team, @spgraph-docs-team to assist. |
|
@dluces , @shweta-bedmutha, is this something that you could help us with? It seems that the issue derives from this issue. It's not clear how to configure a specific property. The customer proposed valid changes but it would be good to use the new property instead of the deprecated property. The SharePoint folks can help make the related documentation clearer, including possible providing better examples._ I appreciate your help. |
|
Thanks @Danipocket. I've asked the product group to look into this. It makes sense that the API supports the new request properties and not just the deprecated one. |


Payload for POST request was incorrect; updated to reflect the correct payload, including grantedToIdentities/application with id AND displayName
Important
Required for API changes:
Add other supporting information, such as a description of the PR changes:
ADD INFORMATION HERE
Important
The following guidance is for Microsoft employees only. Community contributors can ignore this message; our content team will manage the status.
After you've created your PR, expand this section for tips and additional instructions.
#feedback-addressedto the pull request.For more information, see the Content review process summary.