Conversation
- Replace discord_invite_link with discord_client_id for proper OAuth URL - Add Discord invite button in notification preferences (auto-generates bot invite URL) - Persistent discordgo session — always online, reconnects on token change - Rich Discord presence with monitor count + large image - Dedup alerts: 1 email per incident (owner only), remove admin backup CC - Skip email channel type — email already sent once
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (5)
📝 WalkthroughWalkthroughThis PR establishes persistent Discord bot session management in the notifications module, refactors incident email alerts to consolidate them at the incident level (owner only), and adds Discord Client ID configuration across settings and UI views to display bot invite links. ChangesDiscord Bot Session Persistence and Alert Flow
Discord Client ID Configuration UI
Estimated code review effort🎯 4 (Complex) | ⏱️ ~45 minutes Poem
✨ Finishing Touches📝 Generate docstrings
🧪 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 |
Changes
Discord Bot - Persistent Session
Discord Invite Button
discord_invite_linksetting withdiscord_client_idAlert Dedup
Files Changed
modules/notifications/module.go— persistent Discord session, presence, dedupmodules/settings/module.go— add discord_client_id default settingweb/src/validations/setting.ts— add discord_client_id schemaweb/src/views/App.Settings.vue— Client ID field, fetch/saveweb/src/views/App.Notifications.vue— Invite Bot button, MessageCircle iconSummary by CodeRabbit
New Features
Bug Fixes