Skip to content

Apply SharePoint Embedded docs review fixes (review 20260717-172402)#10960

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-172402)#10960
dluces wants to merge 1 commit into
SharePoint:mainfrom
dluces:user/dluces/fix_review_spe

Conversation

@dluces

@dluces dluces commented Jul 21, 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 150 fixes across all severities to 49 files under docs/embedded/ from a full multi-agent docs review (style, accuracy, code) of all 51 embedded articles.

High-severity fixes:

  • limits-calling-patterns.md: "partner tenant" -> "developer tenant" (glossary term).
  • security-compliance-governance.md: spell out "line-of-business (LOB)" on first use.
  • billing-meters.md: reconcile prose meter count with the table (four generally available meters plus one private preview agent message meter).

Medium/low/info fixes: sentence-case titles and headings, glossary and terminology consistency (generic container/container type casing, multitenant, pay-as-you-go), acronym expansions, curly->straight apostrophes, passive->active voice, link-text casing, and evidence-backed role/accuracy corrections.

Apply 150 fixes across all severities to 49 files under docs/embedded/ from a
full multi-agent docs review (style, accuracy, code) of all 51 embedded articles.

High-severity fixes:
- limits-calling-patterns.md: "partner tenant" -> "developer tenant" (glossary term).
- security-compliance-governance.md: spell out "line-of-business (LOB)" on first use.
- billing-meters.md: reconcile prose meter count with the table (four generally
  available meters plus one private preview agent message meter).

Medium/low/info fixes: sentence-case titles and headings, glossary and terminology
consistency (generic container/container type casing, multitenant, pay-as-you-go),
acronym expansions, curly->straight apostrophes, passive->active voice, link-text
casing, and evidence-backed role/accuracy corrections.

Deferred (not applied): the "creating a container type requires no admin role"
product/doc conflict (create-container-type.md, prepare-customer-installation.md,
authentication-permissions.md, container-types-containers.md) awaits human
adjudication; ms.author/ai-usage frontmatter changes need a repo-wide decision.

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 9a717cc:

💡 Validation status: suggestions

File Status Preview URL Details
docs/embedded/build/configure-authentication-authorization.md 💡Suggestion View Details
docs/embedded/build/manage-files.md 💡Suggestion View 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/install-sharepoint-embedded-app.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/admin/setup-billing-microsoft-365-admin-center.md ✅Succeeded View
docs/embedded/build/agent-experiences.md ✅Succeeded View
docs/embedded/build/archive-restore-containers.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/migrate-azure-blob-storage.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/respond-to-changes-webhooks.md ✅Succeeded View

This comment lists only the first 25 files in the pull request.

docs/embedded/build/configure-authentication-authorization.md

  • Line 175, Column 118: [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/filestoragecontainertype-post-permissions?view=graph-rest-beta
  • Line 178, Column 23: [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/filestoragecontainertype-post-permissions?view=graph-rest-beta
  • Line 179, Column 26: [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/filestoragecontainertype-delete-permissions?view=graph-rest-beta
  • Line 180, Column 24: [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/filestoragecontainertype-get?view=graph-rest-beta
  • Line 180, Column 138: [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/filestoragecontainertype-list-permissions?view=graph-rest-beta

docs/embedded/build/manage-files.md

  • Line 154, Column 3: [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/filestoragecontainer-restore-recyclebinitem?view=graph-rest-beta

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.

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