docs: Microsoft Teams (App) alert integration#2286
Conversation
|
👋 @MikaKerman |
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
📝 WalkthroughWalkthroughA Microsoft Teams App alert documentation page was added with setup, routing, commands, supported channels, and disconnect instructions. The legacy webhook page was marked deprecated, and the new page was added to the documentation navigation. 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 |
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
eb97757 to
495c2a5
Compare
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>
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>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Add YouTube demo video embed - Replace connected-dialog screenshot with updated one showing default channel picker - Move default channel selection into the connect step - Keep alert rules as a separate final step Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
495c2a5 to
e18023b
Compare
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