Conversation
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 |
|
Learn Build status updates of commit 53cc837: ✅ Validation status: passed
For more details, please refer to the build report. |
There was a problem hiding this comment.
Pull request overview
Adds documentation for enhancing SharePoint Embedded’s PDF preview experience by appending embed query parameters to a DriveItem webUrl in the existing “Using File Previews” tutorial.
Changes:
- Updated
ms.dateto the current revision date. - Added a new “PDF Preview” section describing an
embedquery parameter mechanism. - Documented two PDF preview parameters: Print (
mpp) and Sticky Notes (mpsn).
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| ```text | ||
| <webUrl>&embed={"<param1>":<value>,"<param2>":<value>} | ||
| ``` |
There was a problem hiding this comment.
The syntax example uses <webUrl>&embed=..., which only works if webUrl already has a ? query string. Consider documenting the correct delimiter (?embed= when there are no existing query params, otherwise &embed=) to avoid readers building invalid URLs.
| ```text | ||
| <webUrl>&embed={"mpp":true} | ||
| ``` |
There was a problem hiding this comment.
The embed value contains JSON with quotes/braces that should be URL-encoded when used in a query string (e.g., via encodeURIComponent), otherwise the example URL is not valid in many clients/browsers. Update the examples to show an encoded embed value (or explicitly state that the JSON must be URL-encoded).
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
Learn Build status updates of commit d8966a9: ✅ Validation status: passed
For more details, please refer to the build report. |
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 |
Category
What's in this Pull Request?
This PR adds a new PDF Preview section to the Using File Previews article, documenting the query parameters available for enhancing the SharePoint Embedded static PDF previewer.
Changes
mpp) — documents the mpp boolean parameter that enables the print icon and Ctrl+P functionalitympsn) — documents the mpsn boolean parameter that shows sticky note content in a PDF