Skip to content

feat: add project names to Slack deployment titles - #10928

Open
dunctk wants to merge 3 commits into
coollabsio:nextfrom
dunctk:agent/slack-project-name
Open

feat: add project names to Slack deployment titles#10928
dunctk wants to merge 3 commits into
coollabsio:nextfrom
dunctk:agent/slack-project-name

Conversation

@dunctk

@dunctk dunctk commented Jul 14, 2026

Copy link
Copy Markdown

Changes

  • Add an optional Slack setting under /notifications/slack to prefix deployment notification titles with the project name.
  • Keep the current notification titles unchanged by default.
  • Cover enabled, disabled, and default-off behavior with focused tests.

Issues

Category

  • Bug fix
  • Improvement
  • New feature
  • Adding new one click service
  • Fixing or updating existing one click service

Preview

The Deployments section now includes a Show Project Name in Title checkbox. It was verified checked and persisted after a hard reload while global Slack delivery remained disabled and the webhook remained blank.

Slack deployment settings with Show Project Name in Title enabled

AI Assistance

  • AI was NOT used to create this PR
  • AI was used (please describe below)

If AI was used:

  • Tools used: Codex
  • How extensively: Extensively for implementation, tests, local environment and browser verification, and PR preparation.

Testing

  • Started a local Coolify development instance with Docker Compose and confirmed /api/health returned OK.
  • Ran php artisan test --compact tests/Feature/SlackNotificationProjectNameTest.php (2 tests, 5 assertions).
  • Ran vendor/bin/pint --format agent on all changed PHP files and vendor/bin/pint --dirty --format agent.
  • Verified /notifications/slack in a real browser: the option saved, remained checked after reload, and produced no browser console errors.

Contributor Agreement

Important

  • I have read and understood the contributor guidelines. If I have failed to follow any guideline, I understand that this PR may be closed without review.
  • I have searched existing issues and pull requests (including closed ones) to ensure this isn't a duplicate.
  • I have tested all the changes thoroughly with a local development instance of Coolify and I am confident that they will work as expected when a maintainer tests them.

@dunctk
dunctk marked this pull request as ready for review July 14, 2026 13:35
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