Document Rocket.Chat provider - #546
Conversation
Adds Rocket.Chat to the provider config example and references in the README, and wires a rocketchat entry into the integration-test config/scaffolding (commented out, following the pattern used for teams/telegram, since no webhook secret is configured in CI yet).
WalkthroughThe README adds Rocket.Chat to the supported providers, documents its configuration, and links to webhook setup instructions. The integration-test provider scaffold includes commented-out Rocket.Chat wiring. ChangesRocket.Chat support
Estimated code review effort: 1 (Trivial) | ~3 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 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 `@README.md`:
- Line 277: Update the Rocket.Chat Incoming Webhook link in the README to use
the current integrations-page section anchor, or remove the stale fragment while
preserving the existing destination.
🪄 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: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: f58e9445-6c3c-4728-a279-498d7359880a
⛔ Files ignored due to path filters (1)
cmd/integration-test/test-config.yamlis excluded by!**/*.yaml
📒 Files selected for processing (2)
README.mdcmd/integration-test/providers.go
| - [Creating Discord webhook](https://support.discord.com/hc/en-us/articles/228383668-Intro-to-Webhooks) | ||
| - [Creating Telegram bot](https://core.telegram.org/bots#3-how-do-i-create-a-bot) | ||
| - [Creating Pushover Token](https://github.com/containrrr/shoutrrr/blob/main/docs/services/pushover.md) | ||
| - [Creating Rocket.Chat Incoming Webhook](https://docs.rocket.chat/docs/integrations#incoming-webhook-script) |
There was a problem hiding this comment.
🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win
Fix the stale fragment in the Rocket.Chat webhook reference.
The page resolves, but #incoming-webhook-script does not match a current section in the Rocket.Chat integrations page. Use the current section anchor or remove the fragment. (docs.rocket.chat)
🤖 Prompt for 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.
In `@README.md` at line 277, Update the Rocket.Chat Incoming Webhook link in the
README to use the current integrations-page section anchor, or remove the stale
fragment while preserving the existing destination.
Summary
rocketchatprovider in the README (features list, provider config example, webhook reference link)rocketchatentry to the integration-test config, commented out inproviders.gofollowing the existing pattern for teams/telegram (no webhook secret configured in CI yet)Depends on #545 for the provider implementation.
Summary by CodeRabbit
New Features
Documentation