Skip to content

Conversation

samantharamon
Copy link
Contributor

@samantharamon samantharamon commented Sep 3, 2025

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 issues. Other issues are also a high priority. 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.

Copy link
Contributor

Learn Build status updates of commit 810ff90:

💡 Validation status: suggestions

File Status Preview URL Details
docs/requirement-sets/outlook/preview-requirement-set/outlook-requirement-set-preview.md 💡Suggestion View (>=common-js || >=outlook-js-1.1 || >=excel-js-1.1 || onenote-js-1.1 || >=powerpoint-js-1.1 || visio-js-1.1 || >=word-js-1.1) Details
docs/manifest/launchevent.md ✅Succeeded View (>=common-js || >=outlook-js-1.1 || >=excel-js-1.1 || onenote-js-1.1 || >=powerpoint-js-1.1 || visio-js-1.1 || >=word-js-1.1)

docs/requirement-sets/outlook/preview-requirement-set/outlook-requirement-set-preview.md

  • Line 98, Column 6: [Suggestion: learn-site-link-broken - See documentation] Link '/javascript/api/outlook/office.messagedecrypteventcompletedoptions?view=outlook-js-preview&preserve-view=true' points to a learn site page that doesn't exist. Check the path or URL and update the link.
  • Line 104, Column 6: [Suggestion: learn-site-link-broken - See documentation] Link '/javascript/api/outlook/office.decryptedmessageattachment?view=outlook-js-preview&preserve-view=true' points to a learn site page that doesn't exist. Check the path or URL and update the link.
  • Line 110, Column 6: [Suggestion: learn-site-link-broken - See documentation] Link '/javascript/api/outlook/office.decryptedmessagebody?view=outlook-js-preview&preserve-view=true' points to a learn site page that doesn't exist. Check the path or URL and update the link.

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.

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 issues. Other issues are also a high priority. 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.

Copy link
Contributor

Learn Build status updates of commit 36f59e3:

💡 Validation status: suggestions

File Status Preview URL Details
docs/requirement-sets/outlook/preview-requirement-set/outlook-requirement-set-preview.md 💡Suggestion View (>=common-js || >=outlook-js-1.1 || >=excel-js-1.1 || onenote-js-1.1 || >=powerpoint-js-1.1 || visio-js-1.1 || >=word-js-1.1) Details
docs/manifest/launchevent.md ✅Succeeded View (>=common-js || >=outlook-js-1.1 || >=excel-js-1.1 || onenote-js-1.1 || >=powerpoint-js-1.1 || visio-js-1.1 || >=word-js-1.1)

docs/requirement-sets/outlook/preview-requirement-set/outlook-requirement-set-preview.md

  • Line 98, Column 6: [Suggestion: learn-site-link-broken - See documentation] Link '/javascript/api/outlook/office.messagedecrypteventcompletedoptions?view=outlook-js-preview&preserve-view=true' points to a learn site page that doesn't exist. Check the path or URL and update the link.
  • Line 104, Column 6: [Suggestion: learn-site-link-broken - See documentation] Link '/javascript/api/outlook/office.decryptedmessageattachment?view=outlook-js-preview&preserve-view=true' points to a learn site page that doesn't exist. Check the path or URL and update the link.
  • Line 110, Column 6: [Suggestion: learn-site-link-broken - See documentation] Link '/javascript/api/outlook/office.decryptedmessagebody?view=outlook-js-preview&preserve-view=true' points to a learn site page that doesn't exist. Check the path or URL and update the link.

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.

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 issues. Other issues are also a high priority. 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.

Copy link
Contributor

Learn Build status updates of commit bc81c33:

💡 Validation status: suggestions

File Status Preview URL Details
docs/manifest/launchevent.md 💡Suggestion View (>=common-js || >=outlook-js-1.1 || >=excel-js-1.1 || onenote-js-1.1 || >=powerpoint-js-1.1 || visio-js-1.1 || >=word-js-1.1) Details
docs/requirement-sets/outlook/preview-requirement-set/outlook-requirement-set-preview.md 💡Suggestion View (>=common-js || >=outlook-js-1.1 || >=excel-js-1.1 || onenote-js-1.1 || >=powerpoint-js-1.1 || visio-js-1.1 || >=word-js-1.1) Details

docs/manifest/launchevent.md

  • Line 69, Column 3: [Suggestion: learn-site-link-broken - See documentation] Link '/office/dev/add-ins/outlook/encryption-decryption?branch=samramon-decryption' points to a learn site page that doesn't exist. Check the path or URL and update the link.

docs/requirement-sets/outlook/preview-requirement-set/outlook-requirement-set-preview.md

  • Line 98, Column 6: [Suggestion: learn-site-link-broken - See documentation] Link '/javascript/api/outlook/office.messagedecrypteventcompletedoptions?view=outlook-js-preview&preserve-view=true' points to a learn site page that doesn't exist. Check the path or URL and update the link.
  • Line 104, Column 6: [Suggestion: learn-site-link-broken - See documentation] Link '/javascript/api/outlook/office.decryptedmessageattachment?view=outlook-js-preview&preserve-view=true' points to a learn site page that doesn't exist. Check the path or URL and update the link.
  • Line 110, Column 6: [Suggestion: learn-site-link-broken - See documentation] Link '/javascript/api/outlook/office.decryptedmessagebody?view=outlook-js-preview&preserve-view=true' points to a learn site page that doesn't exist. Check the path or URL and update the link.

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.

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 issues. Other issues are also a high priority. 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.

Copy link
Contributor

Learn Build status updates of commit 24403c2:

💡 Validation status: suggestions

File Status Preview URL Details
docs/manifest/launchevent.md 💡Suggestion View (>=common-js || >=outlook-js-1.1 || >=excel-js-1.1 || onenote-js-1.1 || >=powerpoint-js-1.1 || visio-js-1.1 || >=word-js-1.1) Details
docs/requirement-sets/outlook/preview-requirement-set/outlook-requirement-set-preview.md 💡Suggestion View (>=common-js || >=outlook-js-1.1 || >=excel-js-1.1 || onenote-js-1.1 || >=powerpoint-js-1.1 || visio-js-1.1 || >=word-js-1.1) Details

docs/manifest/launchevent.md

  • Line 69, Column 3: [Suggestion: learn-site-link-broken - See documentation] Link '/office/dev/add-ins/outlook/encryption-decryption' points to a learn site page that doesn't exist. Check the path or URL and update the link.

docs/requirement-sets/outlook/preview-requirement-set/outlook-requirement-set-preview.md

  • Line 98, Column 6: [Suggestion: learn-site-link-broken - See documentation] Link '/javascript/api/outlook/office.messagedecrypteventcompletedoptions?view=outlook-js-preview&preserve-view=true' points to a learn site page that doesn't exist. Check the path or URL and update the link.
  • Line 104, Column 6: [Suggestion: learn-site-link-broken - See documentation] Link '/javascript/api/outlook/office.decryptedmessageattachment?view=outlook-js-preview&preserve-view=true' points to a learn site page that doesn't exist. Check the path or URL and update the link.
  • Line 110, Column 6: [Suggestion: learn-site-link-broken - See documentation] Link '/javascript/api/outlook/office.decryptedmessagebody?view=outlook-js-preview&preserve-view=true' points to a learn site page that doesn't exist. Check the path or URL and update the link.

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.

Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This pull request adds documentation for new decryption APIs in the Outlook preview requirement set, enabling add-ins to decrypt encrypted messages and their attachments through event-based activation.

  • Added comprehensive documentation for message decryption functionality including new objects, events, and methods
  • Updated the LaunchEvent manifest element to support the OnMessageRead event with HeaderName attribute for encryption
  • Added reference to new encryption/decryption tutorial documentation

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
docs/requirement-sets/outlook/preview-requirement-set/outlook-requirement-set-preview.md Added detailed documentation for decryption APIs including events, objects, and methods
docs/manifest/launchevent.md Updated LaunchEvent documentation to include OnMessageRead event support and expanded HeaderName attribute usage

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@samantharamon samantharamon merged commit f203153 into main Oct 10, 2025
3 checks passed
@samantharamon samantharamon deleted the samramon-decryption branch October 10, 2025 01:43
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.

2 participants