Skip to content

Clarify behavior when both FSC.Manage.All and FSC.Selected are granted for admin and nonadmin#10763

Merged
andrewconnell merged 13 commits intoSharePoint:mainfrom
lilealdai:bothscopes
Apr 24, 2026
Merged

Clarify behavior when both FSC.Manage.All and FSC.Selected are granted for admin and nonadmin#10763
andrewconnell merged 13 commits intoSharePoint:mainfrom
lilealdai:bothscopes

Conversation

@lilealdai
Copy link
Copy Markdown
Contributor

@lilealdai lilealdai commented Apr 10, 2026 •
edited
Loading

Category

  • Content fix
  • New article

What's in this Pull Request?

  • Introduce FileStorageContainer.Manage.All alongside FileStorageContainer.Selected as the two supported Microsoft Graph permissions for delegated access.
  • Added a table describing how effective access varies based on the permissions granted to the application and whether
    the user is an administrator.
  • Fix a couple of incorrect references to filestoragecontainertypemanageall

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

learn-build-service-prod Bot commented Apr 10, 2026

Learn Build status updates of commit 3664737:

✅ Validation status: passed

File Status Preview URL Details
docs/embedded/development/auth.md ✅Succeeded View

For more details, please refer to the build report.

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

learn-build-service-prod Bot commented Apr 10, 2026

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

vhontovyy-MS
vhontovyy-MS reviewed Apr 13, 2026
View reviewed changes
Comment thread docs/embedded/development/auth.md
vhontovyy-MS
vhontovyy-MS reviewed Apr 13, 2026
View reviewed changes
Comment thread docs/embedded/development/auth.md Outdated
vhontovyy-MS
vhontovyy-MS reviewed Apr 13, 2026
View reviewed changes
Comment thread docs/embedded/development/auth.md Outdated
vhontovyy-MS
vhontovyy-MS reviewed Apr 13, 2026
View reviewed changes
Comment thread docs/embedded/development/auth.md Outdated
vhontovyy-MS
vhontovyy-MS reviewed Apr 13, 2026
View reviewed changes
Comment thread docs/embedded/development/auth.md
vhontovyy-MS
vhontovyy-MS reviewed Apr 13, 2026
View reviewed changes
Comment thread docs/embedded/development/auth.md
vhontovyy-MS
vhontovyy-MS reviewed Apr 13, 2026
View reviewed changes
Comment thread docs/embedded/development/auth.md Outdated
@learn-build-service-prod
Copy link
Copy Markdown
Contributor

learn-build-service-prod Bot commented Apr 13, 2026

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

learn-build-service-prod Bot commented Apr 13, 2026

Learn Build status updates of commit 9a2227e:

✅ Validation status: passed

File Status Preview URL Details
docs/embedded/development/auth.md ✅Succeeded View

For more details, please refer to the build report.

@lilealdai lilealdai marked this pull request as ready for review April 13, 2026 22:35
@learn-build-service-prod
Copy link
Copy Markdown
Contributor

learn-build-service-prod Bot commented Apr 13, 2026

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

learn-build-service-prod Bot commented Apr 13, 2026

Learn Build status updates of commit 640bb6e:

✅ Validation status: passed

File Status Preview URL Details
docs/embedded/development/auth.md ✅Succeeded View

For more details, please refer to the build report.

@lilealdai lilealdai requested a review from vhontovyy-MS April 13, 2026 22:45
vhontovyy-MS
vhontovyy-MS reviewed Apr 13, 2026
View reviewed changes
Comment thread docs/embedded/development/auth.md Outdated
vhontovyy-MS
vhontovyy-MS reviewed Apr 13, 2026
View reviewed changes
Comment thread docs/embedded/development/auth.md Outdated
vhontovyy-MS
vhontovyy-MS reviewed Apr 13, 2026
View reviewed changes
Comment thread docs/embedded/development/auth.md Outdated
vhontovyy-MS
vhontovyy-MS reviewed Apr 13, 2026
View reviewed changes
Comment thread docs/embedded/development/auth.md Outdated
vhontovyy-MS
vhontovyy-MS reviewed Apr 13, 2026
View reviewed changes
Comment thread docs/embedded/development/auth.md Outdated
vhontovyy-MS
vhontovyy-MS reviewed Apr 13, 2026
View reviewed changes
Comment thread docs/embedded/development/auth.md Outdated
vhontovyy-MS
vhontovyy-MS reviewed Apr 13, 2026
View reviewed changes
Comment thread docs/embedded/development/auth.md Outdated
@learn-build-service-prod
Copy link
Copy Markdown
Contributor

learn-build-service-prod Bot commented Apr 13, 2026

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

learn-build-service-prod Bot commented Apr 13, 2026

Learn Build status updates of commit 6438db9:

✅ Validation status: passed

File Status Preview URL Details
docs/embedded/development/auth.md ✅Succeeded View

For more details, please refer to the build report.

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

learn-build-service-prod Bot commented Apr 16, 2026

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

learn-build-service-prod Bot commented Apr 16, 2026

Learn Build status updates of commit a01f5b2:

✅ Validation status: passed

File Status Preview URL Details
docs/embedded/development/auth.md ✅Succeeded View

For more details, please refer to the build report.

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

learn-build-service-prod Bot commented Apr 16, 2026

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

learn-build-service-prod Bot commented Apr 16, 2026

Learn Build status updates of commit 1a5911b:

✅ Validation status: passed

File Status Preview URL Details
docs/embedded/development/auth.md ✅Succeeded View

For more details, please refer to the build report.

@dluces dluces marked this pull request as ready for review April 17, 2026 16:23
vhontovyy-MS
vhontovyy-MS reviewed Apr 17, 2026
View reviewed changes
Comment thread docs/embedded/development/auth.md
vhontovyy-MS
vhontovyy-MS reviewed Apr 17, 2026
View reviewed changes
Comment thread docs/embedded/development/auth.md Outdated
@learn-build-service-prod
Copy link
Copy Markdown
Contributor

learn-build-service-prod Bot commented Apr 17, 2026

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

learn-build-service-prod Bot commented Apr 17, 2026

Learn Build status updates of commit 30782e6:

✅ Validation status: passed

File Status Preview URL Details
docs/embedded/development/auth.md ✅Succeeded View

For more details, please refer to the build report.

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

learn-build-service-prod Bot commented Apr 18, 2026

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

learn-build-service-prod Bot commented Apr 18, 2026

Learn Build status updates of commit beede5c:

✅ Validation status: passed

File Status Preview URL Details
docs/embedded/development/auth.md ✅Succeeded View

For more details, please refer to the build report.

@andrewconnell andrewconnell self-requested a review April 24, 2026 19:40
@andrewconnell andrewconnell added the pr:in-review Actively reviewing pull request label Apr 24, 2026
@andrewconnell
docs: revert ms.date and minor formatting tweaks on auth article
564ce23 
- revert `ms.date` to original publication date; rendering engine
  shows last modified date dynamically from git history
- add blank line after intro sentence in access on behalf of a user
  section
- replace "as well as" with "and" in admin capabilities list
- add blank lines around nested list in what's next section
andrewconnell
andrewconnell approved these changes Apr 24, 2026
View reviewed changes
Comment thread docs/embedded/development/auth.md Outdated
title: SharePoint Embedded Authentication and Authorization
description: This article describes the authentication and authorization model for SharePoint Embedded applications.
ms.date: 04/02/2026
ms.date: 04/16/2026
Copy link
Copy Markdown
Collaborator

@andrewconnell andrewconnell Apr 24, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • no need to update the ms.date anymore: the site build engine now uses the git history to determine when it was last updated; this value should be the date it was originally created

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

learn-build-service-prod Bot commented Apr 24, 2026

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

learn-build-service-prod Bot commented Apr 24, 2026

Learn Build status updates of commit 564ce23:

✅ Validation status: passed

File Status Preview URL Details
docs/embedded/development/auth.md ✅Succeeded View

For more details, please refer to the build report.

@andrewconnell andrewconnell added pr:ready-to-merge Ready to merge PR and removed pr:in-review Actively reviewing pull request labels Apr 24, 2026
@andrewconnell andrewconnell merged commit d2627bb into SharePoint:main Apr 24, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dluces dluces dluces approved these changes

@javieralvarezchiang javieralvarezchiang javieralvarezchiang left review comments

@andrewconnell andrewconnell andrewconnell approved these changes

@vhontovyy-MS vhontovyy-MS vhontovyy-MS approved these changes

Assignees

@andrewconnell andrewconnell

Labels

pr:ready-to-merge Ready to merge PR

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@lilealdai @dluces @andrewconnell @javieralvarezchiang @vhontovyy-MS