Webhook system for deployment notifications #2379
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Merged upstream canary branch to get latest v0.24.10 - Updated Node.js to 20.16.0 as required - Fixed database schema with migrations - Resolved dependency issues
- Add webhook and webhook_delivery database tables - Implement complete webhook service with CRUD operations - Add webhook triggering and delivery system - Implement retry mechanism with exponential backoff - Add template formatters for Slack and n8n - Include HMAC signature generation for security - Add delivery logging with 100-entry limit per webhook
- Add webhook triggering for deployment.started event in createDeployment - Add webhook triggering for deployment.started event in createDeploymentCompose - Enhance updateDeploymentStatus to trigger success/failed/cancelled webhooks - Include full deployment context in webhook payloads - Fire webhooks asynchronously to avoid blocking deployments
- Create webhook router with full CRUD operations - Add endpoints for webhook testing and delivery history - Include batch operations and statistics endpoints - Add webhook toggle functionality - Register webhook router in main API router
- Create webhook listing component with status indicators - Add webhook form with support for Slack, n8n, and generic templates - Implement webhook deliveries modal to show webhook history - Add webhook management to application settings page - Include test webhook functionality and delivery statistics - Support enable/disable toggle and batch operations
- Add German (de) translation - Add Spanish (es) translation - Add French (fr) translation - Add Portuguese (pt-br) translation - Add Japanese (ja) translation - Add Chinese Simplified (zh-Hans) translation - Add Russian (ru) translation - Add Italian (it) translation
- Add webhook and webhook_delivery database tables - Implement complete webhook service with CRUD operations - Add webhook triggering and delivery system - Implement retry mechanism with exponential backoff - Add template formatters for Slack and n8n - Include HMAC signature generation for security - Add delivery logging with 100-entry limit per webhook
- Remove remaining i18n references in webhook components - Fix DOM nesting issue in delivery history table by removing Collapsible wrapper - Use React.Fragment with key prop to fix React warning - Fix modal closing issue by using onSelect with preventDefault - Replace hardcoded English strings throughout webhook UI
These test files are already in .gitignore and should not be tracked.
…issues - Split monolithic webhook components into modular, reusable pieces - Created standalone WebhookForm component for create/edit operations - Created dedicated WebhookHistoryModal for delivery history - Created separate WebhookEditModal and WebhookCreateModal - Fixed modal closing issues by removing conditional rendering - Created custom WebhookDialog component without overscroll-contain issue - Improved state management and separation of concerns - Fixed all linting and TypeScript errors This resolves the modal closing and scrolling issues that were preventing proper webhook management functionality. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
- Implement comprehensive n8n webhook template with flattened payload structure - Add Slack Block Kit template with rich formatting and interactive components - Create template configuration forms for both platforms - Add template preview component with visual Slack message preview - Implement template factory pattern for extensibility - Add validation schemas for platform-specific configurations - Create enhanced webhook service layer for integration fix: remove automatic webhook secret generation - Remove backend auto-generation of webhook secrets - Fix form to properly handle undefined secrets - Update edit modal to preserve null/undefined secret state - Ensure users have full control over webhook secret management 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
…webhooks' into feature/deployment-webhooks
Siumauricio
requested changes
Aug 17, 2025
Siumauricio
requested changes
Aug 31, 2025
Comment on lines
24
to
29
| export const WebhookCreateModal = ({ | ||
| applicationId, | ||
| composeId, | ||
| trigger, | ||
| onSuccess, | ||
| }: WebhookCreateModalProps) => { |
There was a problem hiding this comment.
Can you move the WebhookCreateModal and WebhookEditModal to handlewebhook? you can see examples where we handle the creation and the update in the same component, you can see the handle-project.tsx as an example
There was a problem hiding this comment.
refactor: consolidate webhook modals into single handle-webhook component
…nent - Create unified HandleWebhook component following handle-project pattern - Replace WebhookCreateModal and WebhookEditModal with single component - Update show-webhooks.tsx to use new HandleWebhook component - Remove deprecated modal components
A-D-E
force-pushed
the
feature/deployment-webhooks
branch
from
9e8bc46 to
8316222
Compare
Siumauricio
requested changes
Sep 1, 2025
Comment on lines
6
to
120
| "fixed inset-0 z-50 bg-black/80 data-[state=open]:animate-in data-[state=closed]:animate-out data-[state=closed]:fade-out-0 data-[state=open]:fade-in-0", | ||
| className, | ||
| )} | ||
| {...props} | ||
| /> | ||
| )); | ||
| WebhookDialogOverlay.displayName = DialogPrimitive.Overlay.displayName; | ||
|
|
||
| const WebhookDialogContent = React.forwardRef< | ||
| React.ElementRef<typeof DialogPrimitive.Content>, | ||
| React.ComponentPropsWithoutRef<typeof DialogPrimitive.Content> | ||
| >(({ className, children, ...props }, ref) => ( | ||
| <WebhookDialogPortal> | ||
| <WebhookDialogOverlay /> | ||
| <DialogPrimitive.Content | ||
| ref={ref} | ||
| className={cn( | ||
| "fixed left-[50%] top-[50%] z-50 grid w-full translate-x-[-50%] translate-y-[-50%] gap-4 border bg-background p-6 shadow-lg duration-200 data-[state=open]:animate-in data-[state=closed]:animate-out data-[state=closed]:fade-out-0 data-[state=open]:fade-in-0 data-[state=closed]:zoom-out-95 data-[state=open]:zoom-in-95 data-[state=closed]:slide-out-to-left-1/2 data-[state=closed]:slide-out-to-top-[48%] data-[state=open]:slide-in-from-left-1/2 data-[state=open]:slide-in-from-top-[48%] sm:rounded-lg", | ||
| "overflow-y-auto", // Simple overflow without overscroll-contain | ||
| className, | ||
| )} | ||
| {...props} | ||
| > | ||
| {children} | ||
| <DialogPrimitive.Close className="absolute right-4 top-4 rounded-sm opacity-70 ring-offset-background transition-opacity hover:opacity-100 focus:outline-none focus:ring-2 focus:ring-ring focus:ring-offset-2 disabled:pointer-events-none data-[state=open]:bg-accent data-[state=open]:text-muted-foreground"> | ||
| <X className="h-4 w-4" /> | ||
| <span className="sr-only">Close</span> | ||
| </DialogPrimitive.Close> | ||
| </DialogPrimitive.Content> | ||
| </WebhookDialogPortal> | ||
| )); | ||
| WebhookDialogContent.displayName = DialogPrimitive.Content.displayName; | ||
|
|
||
| const WebhookDialogHeader = ({ | ||
| className, | ||
| ...props | ||
| }: React.HTMLAttributes<HTMLDivElement>) => ( | ||
| <div | ||
| className={cn( | ||
| "flex flex-col space-y-1.5 text-center sm:text-left", | ||
| className, | ||
| )} | ||
| {...props} | ||
| /> | ||
| ); | ||
| WebhookDialogHeader.displayName = "WebhookDialogHeader"; | ||
|
|
||
| const WebhookDialogFooter = ({ | ||
| className, | ||
| ...props | ||
| }: React.HTMLAttributes<HTMLDivElement>) => ( | ||
| <div | ||
| className={cn( | ||
| "flex flex-col-reverse sm:flex-row sm:justify-end sm:space-x-2", | ||
| className, | ||
| )} | ||
| {...props} | ||
| /> | ||
| ); | ||
| WebhookDialogFooter.displayName = "WebhookDialogFooter"; | ||
|
|
||
| const WebhookDialogTitle = React.forwardRef< | ||
| React.ElementRef<typeof DialogPrimitive.Title>, | ||
| React.ComponentPropsWithoutRef<typeof DialogPrimitive.Title> | ||
| >(({ className, ...props }, ref) => ( | ||
| <DialogPrimitive.Title | ||
| ref={ref} | ||
| className={cn( | ||
| "text-lg font-semibold leading-none tracking-tight", | ||
| className, | ||
| )} | ||
| {...props} | ||
| /> | ||
| )); | ||
| WebhookDialogTitle.displayName = DialogPrimitive.Title.displayName; | ||
|
|
||
| const WebhookDialogDescription = React.forwardRef< | ||
| React.ElementRef<typeof DialogPrimitive.Description>, | ||
| React.ComponentPropsWithoutRef<typeof DialogPrimitive.Description> | ||
| >(({ className, ...props }, ref) => ( | ||
| <DialogPrimitive.Description | ||
| ref={ref} | ||
| className={cn("text-sm text-muted-foreground", className)} | ||
| {...props} | ||
| /> | ||
| )); | ||
| WebhookDialogDescription.displayName = DialogPrimitive.Description.displayName; | ||
|
|
||
| export { | ||
| WebhookDialog, | ||
| WebhookDialogPortal, | ||
| WebhookDialogOverlay, | ||
| WebhookDialogClose, | ||
| WebhookDialogTrigger, | ||
| WebhookDialogContent, | ||
| WebhookDialogHeader, | ||
| WebhookDialogFooter, | ||
| WebhookDialogTitle, | ||
| WebhookDialogDescription, | ||
| }; |
There was a problem hiding this comment.
I'm wondering why we need to duplicate the base shadcn ui components for this? I don't see much differences with the components we already have
Siumauricio
requested changes
Sep 1, 2025
apps/dokploy/components/dashboard/application/webhooks/handle-webhook.tsx
Show resolved
Hide resolved
- Deleted 0106_mysterious_dracula.sql and 0106_nasty_silver_surfer.sql files containing webhook and webhook_delivery table definitions. - Updated _journal.json to remove references to the deleted SQL files. - Removed 0106_snapshot.json file as it is no longer needed.
…key constraints - Created new SQL files for the 'webhook' and 'webhook_delivery' tables. - Added columns and constraints to establish relationships between these tables and existing application and compose tables. - Updated the journal to include the new migration version.
…bhooks and installation scripts
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This MR implements a complete webhook system that enables real-time deployment notifications and integrations with external services.
🚀 Features Added
🔧 Technical Implementation
webhookandwebhook_deliveriestables📚 Documentation
🎯 Use Cases
✅ Testing
🔒 Security
This implementation provides a solid foundation for deployment automation and team notifications while maintaining security and reliability standards.