feat(notifications): add Microsoft Teams integration for notifications#3729
feat(notifications): add Microsoft Teams integration for notifications#3729Siumauricio merged 2 commits intocanaryfrom
Conversation
- Introduced support for Microsoft Teams notifications, including the ability to create, update, and test connections for Teams notifications. - Updated the notification schema to include Teams as a notification type. - Added Teams icon and UI components for handling Teams notifications in the dashboard. - Implemented backend logic for creating and updating Teams notifications, along with necessary database schema changes. - Enhanced existing notification functionalities to support Teams notifications across various events (e.g., build success, failure, database backups).
|
|
||
| export interface TeamsAdaptiveCardMessage { | ||
| title: string; | ||
| themeColor?: string; |
There was a problem hiding this comment.
Unused interface field themeColor
The themeColor property is defined on the TeamsAdaptiveCardMessage interface but is never read or used anywhere in sendTeamsNotification. If you don't plan to use it, consider removing it to keep the interface clean. If it's intended for future use, adding a comment would clarify intent.
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
| const { discord, telegram, slack, custom, lark, pushover, teams } = | ||
| notification; |
There was a problem hiding this comment.
Indentation on continuation line
The continuation line notification; should be indented further (e.g., one extra tab level) to clearly indicate it's a continuation of the destructuring statement.
| const { discord, telegram, slack, custom, lark, pushover, teams } = | |
| notification; | |
| const { discord, telegram, slack, custom, lark, pushover, teams } = | |
| notification; |
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
What is this PR about?
Please describe in a short paragraph what this PR is about.
Checklist
Before submitting this PR, please make sure that:
canarybranch.Issues related (if applicable)
closes #2783
Screenshots (if applicable)
Greptile Summary
This PR adds Microsoft Teams as a new notification provider in Dokploy, implementing the full stack: database schema/migration, backend services (create, update, test), TRPC router endpoints, and frontend UI components. The Teams integration uses the Adaptive Card format, which is the current standard for Teams webhooks.
teamsdatabase table withteamsIdandwebhookUrlcolumns, with proper foreign key and cascade delete on thenotificationtablesendTeamsNotificationusing the Adaptive Card payload format with support for title, facts, and action buttonsConfidence Score: 4/5
Last reviewed commit: d516332