Skip to content

Apply SharePoint Embedded docs review fixes (review 20260717-223227)#10958

Merged
andrewconnell merged 1 commit into
SharePoint:mainfrom
dluces:user/dluces/fix_review_spe
Jul 21, 2026
Merged

Apply SharePoint Embedded docs review fixes (review 20260717-223227)#10958
andrewconnell merged 1 commit into
SharePoint:mainfrom
dluces:user/dluces/fix_review_spe

Conversation

@dluces

@dluces dluces commented Jul 18, 2026

Copy link
Copy Markdown
Contributor

Category

  • Content fix
  • New article

Related issues

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

What's in this Pull Request?

Apply 88 high-confidence fixes across 39 files under docs/embedded/ from the multi-agent docs review:

  • Correct billing setup role to Global Administrator (SharePoint Embedded Administrator can't configure billing) and fix "three" -> "four" billing meters.
  • Fix container metadata schema-change permission: owners and managers (not owners only) can create/update/delete columns.
  • Add missing conventions to sharepoint-embedded-mcp-server.md (Applies to line, agent metadata block, fixed links).
  • Apply M365/Acrolinx style fixes: sentence-case headings, admin center casing, terminology consistency, and typo corrections.

Apply 88 high-confidence fixes across 39 files under docs/embedded/ from the
multi-agent docs review:

- Correct billing setup role to Global Administrator (SharePoint Embedded
  Administrator can't configure billing) and fix "three" -> "four" billing meters.
- Fix container metadata schema-change permission: owners and managers (not
  owners only) can create/update/delete columns.
- Add missing conventions to sharepoint-embedded-mcp-server.md (Applies to line,
  agent metadata block, ms.reviewer, fixed links).
- Apply M365/Acrolinx style fixes: sentence-case headings, admin center casing,
  terminology consistency, and typo corrections.

Container-type "no admin role required" claims were intentionally left unchanged
pending human adjudication of the product/doc conflict.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 4d4d88de-108b-4260-8798-529f5b1b0dc3
@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.

@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 9c7027d:

✅ Validation status: passed

File Status Preview URL Details
docs/embedded/admin/admin-overview.md ✅Succeeded View
docs/embedded/admin/apply-security-compliance-controls.md ✅Succeeded View
docs/embedded/admin/consuming-tenant-admin.md ✅Succeeded View
docs/embedded/admin/create-apps-powershell.md ✅Succeeded View
docs/embedded/admin/create-apps-sharepoint-admin-center.md ✅Succeeded View
docs/embedded/admin/grant-admin-consent-permissions.md ✅Succeeded View
docs/embedded/admin/manage-containers-powershell.md ✅Succeeded View
docs/embedded/admin/manage-containers-sharepoint-admin-center.md ✅Succeeded View
docs/embedded/admin/monitor-usage-billing-cost.md ✅Succeeded View
docs/embedded/admin/review-audit-events.md ✅Succeeded View
docs/embedded/build/archive-restore-containers.md ✅Succeeded View
docs/embedded/build/configure-authentication-authorization.md ✅Succeeded View
docs/embedded/build/container-metadata.md ✅Succeeded View
docs/embedded/build/create-container-type.md ✅Succeeded View
docs/embedded/build/create-manage-containers.md ✅Succeeded View
docs/embedded/build/fluid-framework.md ✅Succeeded View
docs/embedded/build/manage-files.md ✅Succeeded View
docs/embedded/build/open-office-files.md ✅Succeeded View
docs/embedded/build/preview-files.md ✅Succeeded View
docs/embedded/build/quickstart-vscode.md ✅Succeeded View
docs/embedded/build/register-application-permissions.md ✅Succeeded View
docs/embedded/build/search-containers-files.md ✅Succeeded View
docs/embedded/build/sharepoint-embedded-mcp-server.md ✅Succeeded View
docs/embedded/overview.md ✅Succeeded View
docs/embedded/plan/app-tenant-architecture.md ✅Succeeded View

This comment lists only the first 25 files in the pull request.
For more details, please refer to the build report.

@andrewconnell andrewconnell self-assigned this Jul 21, 2026
@andrewconnell andrewconnell added the pr:in-review Actively reviewing pull request label Jul 21, 2026
@andrewconnell
andrewconnell self-requested a review July 21, 2026 10:49
@andrewconnell andrewconnell added pr:ready-to-merge Ready to merge PR and removed pr:in-review Actively reviewing pull request labels Jul 21, 2026
@andrewconnell
andrewconnell merged commit 688fbe1 into SharePoint:main Jul 21, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr:ready-to-merge Ready to merge PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants