Skip to content

Clarify exceptional access pattern for SPE requiring user licenses #10262

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

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

dluces
Copy link
Contributor

@dluces dluces commented May 15, 2025

Category

  • Content fix
  • New article

What's in this Pull Request?

Some SPE APIs still require a user licence to work in user-delegated mode. These are now documented in the Auth article.

@dluces dluces requested review from cindylay and tonchan-msft May 15, 2025 02:36
Copy link
Contributor

Learn Build status updates of commit 186f6e0:

💡 Validation status: suggestions

File Status Preview URL Details
docs/embedded/development/auth.md 💡Suggestion View Details

docs/embedded/development/auth.md

  • Line 97, Column 5: [Suggestion: preserve-view-not-set - See documentation] You've pinned this link to a specific version of content with the view parameter. It's recommended not to pin a version unless that version is A) not the default view and B) the context is about that version specifically. To proceed with pinning a version add the &preserve-view=true to the URL. Otherwise, remove the view parameter. URL: /graph/api/filestorage-list-containers?view=graph-rest-1.0&tabs=http

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.

For any questions, please:

@@ -87,6 +88,14 @@ To use [Microsoft Search](/microsoftsearch/overview-microsoft-search) on SharePo
> [!NOTE]
> Microsoft Search support for SharePoint Embedded content is in Preview and is subject to change. The access requirements for Microsoft Search on SharePoint Embedded content will align with the SharePoint Embedded authorization model in the future. Stay tuned.

#### Operations that require a user license

SharePoint Embedded is designed to work without the need for end-users to have any kind of Microsoft 365 product licenses assigned to them. However, there are certain operations that still require the end-user to have a SharePoint product license assigned to them.
Copy link
Contributor

Choose a reason for hiding this comment

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

i am not sure if this is technically correct per se. A user has a MySite created but no longer have a license are technically not restrained. Let me run point to put together a more descriptive statement here

Copy link
Contributor

Choose a reason for hiding this comment

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

Change this to "SharePoint Embedded is designed to work without the need for end-users to have any kind of Microsoft 365 product licenses assigned to them. However, there are certain operations that are not available to users without an ODB site."

@andrewconnell andrewconnell self-assigned this May 15, 2025
@andrewconnell andrewconnell added the pr:in-review Actively reviewing pull request label May 15, 2025
@andrewconnell
Copy link
Collaborator

Because @cindylay was requested to review, that indicates to us it's not ready for final review & merge. So, switching this to draft mode. Once @cindylay completes the review, can switch it from DRAFT > OPEN indicating it's ready for our review for merging to live docs.

@andrewconnell andrewconnell marked this pull request as draft May 15, 2025 14:55
@andrewconnell andrewconnell removed the pr:in-review Actively reviewing pull request label May 15, 2025
@andrewconnell andrewconnell removed their assignment May 15, 2025
@tonchan-msft tonchan-msft removed the request for review from cindylay May 15, 2025 17:30
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.

3 participants