[Outlook] (online meeting) Apply feedback #5549
Conversation
samantharamon
commented
Dec 16, 2025
samantharamon
commented
- Simplifies the testing guidance.
- Replaces server links in the sample manifest, so that users can test the add-in after creating the Yo Office project.
- Clarifies prerequisites to include the Join meeting button.
- Clarifies guidance on how to run an add-in that uses the unified manifest in Outlook on Mac and on mobile.
- Adds screenshots for each supported client.
|
Learn Build status updates of commit d08282d: 💡 Validation status: suggestions
docs/images/outlook-owa-new-win-create-meeting.png
docs/images/outlook-owa-new-win-join-meeting.png
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. |
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 |
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 68ef5fe: ✅ Validation status: passed
For more details, please refer to the build report. |
There was a problem hiding this comment.
Pull request overview
This PR updates the Outlook online-meeting provider add-in documentation to improve usability and clarity. The changes simplify testing guidance by replacing production-style URLs with localhost addresses, clarify prerequisites for the Join button feature, improve guidance for unified manifest support across platforms, and add comprehensive screenshots for all supported clients.
- Updated manifest examples to use
localhost:3000instead ofcontoso.comfor easier local testing - Restructured and clarified the testing section with platform-specific guidance and screenshots
- Split "Register your online-meeting template" into a dedicated "Implement the Join button" section
- Added NOTE callout explaining unified manifest limitations on Mac and mobile platforms
Reviewed changes
Copilot reviewed 1 out of 7 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| docs/outlook/online-meeting.md | Updated URLs to localhost, improved testing guidance, added screenshots, restructured Join button documentation, fixed capitalization in tooltip text |
| docs/images/*.png | Added new screenshot images for Outlook on web, Windows (classic and new), Mac, and mobile platforms |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Elizabeth Samuel <[email protected]>
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 1c32d4e: ✅ 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 |
|
Learn Build status updates of commit 17fbf1a: ✅ Validation status: passed
For more details, please refer to the build report. |
