Skip to content

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

Open
dluces wants to merge 1 commit into
SharePoint:mainfrom
dluces:user/dluces/fix_review_spe
Open

Apply SharePoint Embedded docs review fixes (review 20260717-223227)#10958
dluces wants to merge 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.

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.

1 participant