feat: add commit author to all deployment notifications#8511
Open
SelimEmre wants to merge 5 commits intocoollabsio:nextfrom
Open
feat: add commit author to all deployment notifications#8511SelimEmre wants to merge 5 commits intocoollabsio:nextfrom
SelimEmre wants to merge 5 commits intocoollabsio:nextfrom
Conversation
There was a problem hiding this comment.
Pull request overview
This PR adds commit author information to deployment notifications across all channels (Discord, Telegram, Pushover, Slack, Email, Webhook). It introduces a new database column commit_author in the application_deployment_queues table and updates both success and failure notification classes to query and display this information when available.
Changes:
- Added database migration to create
commit_authorcolumn - Updated ApplicationDeploymentQueue model to include
commit_authorin OpenAPI schema - Modified DeploymentSuccess and DeploymentFailed notification classes to fetch and display commit author across all notification methods
- Updated Blade email templates to display commit author when available
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| database/migrations/2025_02_20_165000_add_commit_author_to_application_deployment_queues.php | Adds nullable string column for commit author after commit_message |
| app/Models/ApplicationDeploymentQueue.php | Updates OpenAPI schema to include commit_author field |
| app/Notifications/Application/DeploymentSuccess.php | Queries deployment and displays commit_author in all notification channels (Discord, Telegram, Pushover, Slack, Email, Webhook) |
| app/Notifications/Application/DeploymentFailed.php | Queries deployment and displays commit_author in all notification channels (Discord, Telegram, Pushover, Slack, Email, Webhook) |
| resources/views/emails/application-deployment-success.blade.php | Adds commit author display in success email template |
| resources/views/emails/application-deployment-failed.blade.php | Adds commit author display in failure email template |
database/migrations/2025_02_20_165000_add_commit_author_to_application_deployment_queues.php
Show resolved
Hide resolved
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
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.
Added the commit_author detail from ApplicationDeploymentQueue to all deployment notification channels (toDiscord, toTelegram, toPushover, toMail, toWebhook) during both successful deployments (DeploymentSuccess.php) and failed deployments (DeploymentFailed.php). Blade email templates were also updated to render the commit author if available.
This ensures that team members receive full visibility into who authored the commit that triggered the deployment directly from the notification channel, regardless of the platform they use to monitor deployments.
Issues
#8510
Category
AI Usage
Steps to Test
Contributor Agreement
Important