docs: Microsoft Teams (App) alert integration#2286
Open
MikaKerman wants to merge 3 commits into
Open
Conversation
Documents the new Teams app (bot) messaging integration: install the Elementary app in Teams, connect via the welcome card / @elementary connect, route alerts to channels with alert rules, bot commands, supported channels, and disconnect (remove the app in Teams). Mirrors the existing Slack / MS Teams (webhook) docs. Screenshots are marked with placeholders (REPLACE_WITH_HOSTED_URL + a comment describing each shot) for the docs team to drop in hosted images. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Contributor
|
👋 @MikaKerman |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (11)
📒 Files selected for processing (3)
✅ Files skipped from review due to trivial changes (3)
📝 WalkthroughWalkthroughA new Microsoft Teams App alert docs page was added with setup, routing, commands, supported channels, and disconnect behavior. The legacy Microsoft Teams webhook page was marked deprecated, and the docs navigation now includes the new page. ChangesMS Teams Alert Documentation
Estimated code review effort: 1 (Trivial) | ~3 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@docs/cloud/integrations/alerts/ms-teams-app.mdx`:
- Around line 14-19: The MS Teams app docs still contain placeholder image
sources in the screenshot frames, so replace every REPLACE_WITH_HOSTED_URL
reference with the real hosted asset URLs or remove the affected <Frame>/<img>
blocks until the screenshots are available. Update the repeated screenshot
sections in the MDX page consistently so the visible examples in the alert
walkthrough all render correctly before publishing.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 2365a114-3dff-4a0b-8b00-a3c9f3ab266a
📒 Files selected for processing (2)
docs/cloud/integrations/alerts/ms-teams-app.mdxdocs/mint.json
The onboarding begins in Elementary (Connect messaging app → Microsoft Teams), which deep-links into Teams to install the app — not from the Teams store. Reorder the steps and screenshot placeholders to match. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
- Add all screenshots for the Teams App integration docs page - Simplify connection flow from 5 steps to 3 (install → connect → route) - Label each step with where the action happens (Elementary vs Teams) - Add deprecation warning to the legacy webhook page - Reorder nav to list Teams App before legacy webhook Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
For the docs team
Adds a new Cloud alert-integration page for the Microsoft Teams app (bot) integration — the new app-based Teams integration that replaces the legacy webhook integration.
What's here
docs/cloud/integrations/alerts/ms-teams-app.mdx— titled "Microsoft Teams (App)".docs/cloud/integrations/alerts/ms-teams.mdx— renamed to "Microsoft Teams (Legacy webhook)" with a deprecation warning pointing to the new app page.docs/mint.json— the new app page now appears before the legacy webhook page.docs/images/teams/.Page structure
@Elementary helpand@Elementary connect, each with a screenshot.Screenshots included
@Elementary helpresponse@Elementary connectresponseChanges from initial draft
Summary by CodeRabbit
New Features
Documentation