-
Notifications
You must be signed in to change notification settings - Fork 466
Update permissions-reference.md #9764
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
Conversation
Clarified the behavior of AdminConsentRequired and its relation to user consent settings and app consent policies.
|
Learn Build status updates of commit ba653ba:
|
| File | Status | Preview URL | Details |
|---|---|---|---|
| concepts/permissions-reference.md | Details |
concepts/permissions-reference.md
- Line 27, Column 356: [Warning: hard-coded-locale - See documentation]
Link 'https://learn.microsoft.com/en-us/entra/identity/enterprise-apps/manage-app-consent-policies' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites. - Line 27, Column 356: [Suggestion: docs-link-absolute - See documentation]
Absolute link 'https://learn.microsoft.com/en-us/entra/identity/enterprise-apps/manage-app-consent-policies' will be broken in isolated environments. Replace with a relative link.
For more details, please refer to the build report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
|
Learn Build status updates of commit 867fdce: ✅ 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.
Hi @tomatsue - Thank you for proposing these changes.
I've pushed two commits to move the note to the permissions overview and not the autogenerated reference to help us keep all conceptual information in one central article.
|
Learn Build status updates of commit 9a3954b: ✅ Validation status: passed
For more details, please refer to the build report. |
|
Learn Build status updates of commit 637e711: ✅ Validation status: passed
For more details, please refer to the build report. |
Clarified the behavior of AdminConsentRequired and its relation to user consent settings and app consent policies.
Important
Required for API changes:
Add other supporting information, such as a description of the PR changes:
Clarified the behavior of AdminConsentRequired and its relation to app consent policies because the policies changed recently.
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.