Skip to content

Conversation

@jansenbe
Copy link
Contributor

Category

  • Content fix
  • New article

What's in this Pull Request?

CSP was announced a while ago without a concrete rollout plan. We now have a locked plan and CSP updates, both of which are included in the updated doc.

@learn-build-service-prod
Copy link
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
Contributor

Learn Build status updates of commit d5ab486:

💡 Validation status: suggestions

File Status Preview URL Details
docs/spfx/content-securty-policy-trusted-script-sources.md 💡Suggestion View Details

docs/spfx/content-securty-policy-trusted-script-sources.md

  • Line 6, Column 12: [Suggestion: ms-author-invalid - See documentation] Invalid value for 'ms.author', 'v-jconnell' is not a valid Microsoft alias.

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.

@learn-build-service-prod
Copy link
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
Contributor

Learn Build status updates of commit d10a269:

✅ Validation status: passed

File Status Preview URL Details
docs/spfx/content-securty-policy-trusted-script-sources.md ✅Succeeded View

For more details, please refer to the build report.

@learn-build-service-prod
Copy link
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
Contributor

Learn Build status updates of commit c2c08d6:

✅ Validation status: passed

File Status Preview URL Details
docs/spfx/content-securty-policy-trusted-script-sources.md ✅Succeeded View

For more details, please refer to the build report.

@learn-build-service-prod
Copy link
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
Contributor

Learn Build status updates of commit 6ea3374:

✅ Validation status: passed

File Status Preview URL Details
docs/spfx/content-securty-policy-trusted-script-sources.md ✅Succeeded View

For more details, please refer to the build report.

@andrewconnell andrewconnell self-assigned this Nov 18, 2025
@andrewconnell andrewconnell added the pr:in-review Actively reviewing pull request label Nov 18, 2025
@andrewconnell andrewconnell self-requested a review November 18, 2025 18:42
Corrected grammatical errors and improved clarity in the documentation regarding Content Security Policy (CSP) in SharePoint Online.

Markdown code in bullets should be 2x indented
@andrewconnell andrewconnell added pr:ready-to-merge Ready to merge PR and removed pr:in-review Actively reviewing pull request labels Nov 18, 2025
@learn-build-service-prod
Copy link
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
Contributor

Learn Build status updates of commit c8eacd9:

✅ Validation status: passed

File Status Preview URL Details
docs/spfx/content-securty-policy-trusted-script-sources.md ✅Succeeded View

For more details, please refer to the build report.

@andrewconnell andrewconnell self-requested a review November 18, 2025 18:51
@andrewconnell andrewconnell merged commit 6092c58 into main Nov 18, 2025
3 checks passed
@jansenbe jansenbe deleted the bjansen/csp-updates branch November 19, 2025 09:06
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.

3 participants