Skip to content

fix(notifications): include remote server in Docker cleanup messages - #5458

Open
dscamargo wants to merge 6 commits into
Dokploy:canaryfrom
dscamargo:feature/cleanup-notification-message
Open

dscamargo wants to merge 6 commits into
Dokploy:canaryfrom
dscamargo:feature/cleanup-notification-message

Conversation

@dscamargo

@dscamargo dscamargo commented Sep 13, 2026

Copy link
Copy Markdown

What is this PR about?

This PR updates Docker cleanup notifications for remote servers to include the server name and ID, making it clear where the cleanup was executed. The server-specific message is now used consistently
across all notification channels.

RetriggerConfidence Score: 4/5

This PR is not safe to merge until cleanup notices escape server names for each channel.

Summary

  • Scheduled cleanup messages include the server name and ID.
  • Manual cleanup uses the same server-specific message.
  • Email subjects and content now show the supplied cleanup message.

Reviews (1) · Last reviewed commit: "fix(notifications): include remote serve..."

Comment thread apps/dokploy/server/utils/docker-cleanup.ts
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant