Apply SharePoint Embedded docs review fixes (review 20260721-145410) - #10973
Draft
dluces wants to merge 1 commit into
Draft
Apply SharePoint Embedded docs review fixes (review 20260721-145410)#10973dluces wants to merge 1 commit into
dluces wants to merge 1 commit into
Conversation
Apply 116 fixes across all severities to 50 articles and llms.txt under docs/embedded/ from a full multi-agent docs review (style, accuracy, code) of all 51 embedded files. Verified accuracy/code fixes: - create-container-type.md: trial per-container storage limit 1 GB -> 200 MB (verified against SPO.Core SPTrialContainerType.cs; the 1 GB constant applies only to standard/default container types). - create-container-type.md and container-types-containers.md: clarify the 25 container-type cap applies to standard (non-trial) types, with one trial type counted separately (up to 26 total), and note 25 is the upper limit. - container-types-containers.md: ContainerTypeID -> Graph casing containerTypeId. - llms.txt: correct two machine-index URL slugs to match real page filenames (container-types-containers, setup-billing-microsoft-365-admin-center). Plus style/terminology/grammar/casing/link fixes across the set. ms.date bumped to 07/21/2026 on every edited file. Deferred (not applied): the "creating a container type requires no admin role" product/doc conflict, ms.author/ai-usage frontmatter (needs repo-wide decision), and unverified quota/date/availability claims lacking a concrete corrected value. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 4d4d88de-108b-4260-8798-529f5b1b0dc3
dluces
commented
Jul 31, 2026
| title: Admin overview | ||
| description: Learn how administrators manage SharePoint Embedded apps, containers, billing, and compliance in Microsoft 365. | ||
| ms.date: 07/13/2026 | ||
| ms.date: 07/21/2026 |
Contributor
Author
There was a problem hiding this comment.
This was generated from a previous version of the skills that updated ms.date. I'll review the contents and fix things manually if the update is useful, otherwise will generate a new one from scratch.
Contributor
|
Learn Build status updates of commit b2181be: 💡 Validation status: suggestionsThis comment lists only the first 25 files in the pull request. docs/embedded/build/configure-authentication-authorization.md
docs/embedded/build/manage-files.md
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. |
Contributor
PoliCheck Scan ReportThe 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 foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Category
Related issues
What's in this Pull Request?
Apply 116 fixes across all severities to 50 articles and llms.txt under docs/embedded/ from a full multi-agent docs review (style, accuracy, code) of all 51 embedded files.
Verified accuracy/code fixes:
Plus style/terminology/grammar/casing/link fixes across the set. ms.date bumped to 07/21/2026 on every edited file.
Deferred (not applied): the "creating a container type requires no admin role" product/doc conflict, ms.author/ai-usage frontmatter (needs repo-wide decision), and unverified quota/date/availability claims lacking a concrete corrected value.