Skip to content

Minor improvements to SharePoint Embedded auth page#10847

Open
dluces wants to merge 3 commits into
SharePoint:mainfrom
dluces:user/dluces/spe_auth_confidential_client_notes
Open

Minor improvements to SharePoint Embedded auth page#10847
dluces wants to merge 3 commits into
SharePoint:mainfrom
dluces:user/dluces/spe_auth_confidential_client_notes

Conversation

@dluces
Copy link
Copy Markdown
Contributor

@dluces dluces commented May 20, 2026

Category

  • Content fix
  • New article

Related issues

  • fixes n/a
  • partially n/a
  • mentioned in n/a

What's in this Pull Request?

Three different changes:

  • Fix overview line to clarify access in user-delegated mode to not be constrained to container membership (specific driveItem access can be granted)
  • Recommend that confidential client applications are used in order to retain control.
  • Remove recommendation to delete FSCTR.Selected from manifest once a container type is registered in a consuming tenant. This was incorrect. Apps need the scope to maintain control over container type registrations.

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

Learn Build status updates of commit 4df8204:

✅ 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

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants