-
Couldn't load subscription status.
- Fork 449
Update site-post-permissions.md #9565
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
there was an error in the permissions that listed the delegated work/school perms twice, and didn't mention the application permissions which are required to create the entry.
|
Learn Build status updates of commit cc58224: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
| |Delegated (work or school account)|Not supported.|Not supported.| | ||
| |Delegated (work or school account)|Sites.FullControl.All|Not available.| | ||
| |Delegated (personal Microsoft account)|Not supported.|Not supported.| | ||
| |Application|Sites.FullControl.All|Not available.| |
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.
@westleyMS, can you confirm that the correction you made is what your workload has on the permissions model?
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.
I am not sure what you mean by that. There is an error in that you can't have that permission as 'not available' but then also set to something else. It is there twice, so this was an error.
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.
Hi @westleyMS, sorry about that. I realize I wasn’t clear enough earlier. Yes, we did have duplicate rows for the Delegated (work or school account) permission, but we don’t recommend updating permissions manually.
So my question is: have you confirmed that your permissions model contains the correct data? Thanks!
|
Hello @@westleyMS! This pull request has been inactive for 2 weeks with the
Note Pull requests that are inactive more than 6 weeks will be automatically closed. |
|
Hello @@westleyMS! This pull request has been inactive for 4 weeks with the
Note Pull requests that are inactive more than 6 weeks will be automatically closed. |
|
Hello @@westleyMS! This pull request has been inactive for 6 weeks with the
Note This pull request will be automatically closed in 7 days if you do not take action. |
there was an error in the permissions that listed the delegated work/school perms twice, and didn't mention the application permissions which are required to create the entry.
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.