Add Mattebridge integration to documentation + adapt Shelly & Somfy tutorials#334
Add Mattebridge integration to documentation + adapt Shelly & Somfy tutorials#334Pierre-Gilles merged 2 commits intomasterfrom
Conversation
📝 WalkthroughWalkthroughA new Matterbridge integration documentation page was added to both English and French versions of the documentation. Existing integration guides (Shelly, Somfy Tahoma) were refactored to reference the new Matterbridge documentation instead of duplicating Docker deployment instructions. The sidebar navigation was updated to include the new integration page. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches
🧪 Generate unit tests (beta)
Comment |
Deploying v4-website with
|
| Latest commit: |
eb7c479
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://31f6d1b1.v4-website.pages.dev |
| Branch Preview URL: | https://mattebridge-integration.v4-website.pages.dev |
There was a problem hiding this comment.
🧹 Nitpick comments (2)
docs/integrations/shelly.md (1)
17-17: Consider updating the intro list to match the simplified section title.The intro mentions "Deploy Matterbridge in a Docker container" but section 1 (line 29) is now titled "Deploy Matterbridge" with Docker details delegated to the Matterbridge guide. Consider aligning them for consistency.
-1. Deploy Matterbridge in a Docker container. +1. Deploy Matterbridge.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/integrations/shelly.md` at line 17, Update the intro numbered list item that currently reads "Deploy Matterbridge in a Docker container" to match the section title "Deploy Matterbridge" so wording is consistent with the section (or alternatively re-title the section to explicitly mention Docker if you want the intro to remain Docker-specific); locate the intro list entry text and the section header "Deploy Matterbridge" in docs/integrations/shelly.md and make the two phrases consistent.i18n/fr/docusaurus-plugin-content-docs/current/integrations/shelly.md (1)
17-17: Consider updating the intro list to match the simplified section title.Same as the English version - the intro mentions Docker deployment details while section 1 now delegates to the Matterbridge guide.
-1. Déployer Matterbridge dans un conteneur Docker. +1. Déployer Matterbridge.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@i18n/fr/docusaurus-plugin-content-docs/current/integrations/shelly.md` at line 17, The intro numbered list currently says "1. Déployer Matterbridge dans un conteneur Docker." which mentions Docker details that mismatch the simplified section title; update that list item in integrations/shelly.md to match the simplified heading by removing Docker-specific wording and instead point to the Matterbridge guide (e.g., "1. Déployer Matterbridge — voir le guide Matterbridge" or similar), ensuring the intro mirrors the English version and doesn't imply Docker-specific instructions.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In `@docs/integrations/shelly.md`:
- Line 17: Update the intro numbered list item that currently reads "Deploy
Matterbridge in a Docker container" to match the section title "Deploy
Matterbridge" so wording is consistent with the section (or alternatively
re-title the section to explicitly mention Docker if you want the intro to
remain Docker-specific); locate the intro list entry text and the section header
"Deploy Matterbridge" in docs/integrations/shelly.md and make the two phrases
consistent.
In `@i18n/fr/docusaurus-plugin-content-docs/current/integrations/shelly.md`:
- Line 17: The intro numbered list currently says "1. Déployer Matterbridge dans
un conteneur Docker." which mentions Docker details that mismatch the simplified
section title; update that list item in integrations/shelly.md to match the
simplified heading by removing Docker-specific wording and instead point to the
Matterbridge guide (e.g., "1. Déployer Matterbridge — voir le guide
Matterbridge" or similar), ensuring the intro mirrors the English version and
doesn't imply Docker-specific instructions.
ℹ️ Review info
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (6)
static/img/docs/en/configuration/matterbridge/mattebridge-activate-integration.pngis excluded by!**/*.pngstatic/img/docs/en/configuration/matterbridge/matterbridge-integration-list.pngis excluded by!**/*.pngstatic/img/docs/en/configuration/shelly/1-matterbridge-logs.pngis excluded by!**/*.pngstatic/img/docs/fr/configuration/matterbridge/mattebridge-activate-integration.pngis excluded by!**/*.pngstatic/img/docs/fr/configuration/matterbridge/matterbridge-integration-list.pngis excluded by!**/*.pngstatic/img/docs/fr/configuration/shelly/1-matterbridge-logs.pngis excluded by!**/*.png
📒 Files selected for processing (7)
docs/integrations/matterbridge.mddocs/integrations/shelly.mddocs/integrations/somfy-tahoma.mdi18n/fr/docusaurus-plugin-content-docs/current/integrations/matterbridge.mdi18n/fr/docusaurus-plugin-content-docs/current/integrations/shelly.mdi18n/fr/docusaurus-plugin-content-docs/current/integrations/somfy-tahoma.mdsidebars.js
Summary by CodeRabbit
Documentation