[PR #817] [MERGED] Add database notifications #988

Closed
opened 2026-02-05 18:00:28 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/pelican-dev/panel/pull/817
Author: @Boy132
Created: 12/11/2024
Status: Merged
Merged: 12/12/2024
Merged by: @Boy132

Base: mainHead: feature/database-notifications


📝 Commits (10+)

  • 091d0b4 add database notifications to all panels
  • e9ab70c add successful param to Installed event
  • 1358c01 add listener for Installed event
  • 6153bee create event for subuser creation
  • c1dde6e add listener for SubUserAdded event
  • 9cbeebc always send Installed event
  • 79eab82 create event for subuser removal
  • 75337ce add listener for SubUserRemoved event
  • f2d676b add prefix to server name
  • d4fa109 remove view action from SubUserRemoved notification

📊 Changes

16 files changed (+176 additions, -45 deletions)

View changed files

📝 app/Events/Server/Installed.php (+1 -1)
app/Events/Server/SubUserAdded.php (+17 -0)
app/Events/Server/SubUserRemoved.php (+18 -0)
📝 app/Http/Controllers/Api/Remote/Servers/ServerInstallController.php (+5 -15)
app/Listeners/Server/ServerInstalledListener.php (+29 -0)
app/Listeners/Server/SubUserAddedListener.php (+29 -0)
app/Listeners/Server/SubUserRemovedListener.php (+17 -0)
📝 app/Notifications/AddedToServer.php (+18 -5)
📝 app/Notifications/RemovedFromServer.php (+18 -5)
📝 app/Notifications/ServerInstalled.php (+16 -7)
📝 app/Providers/Filament/AdminPanelProvider.php (+1 -0)
📝 app/Providers/Filament/AppPanelProvider.php (+1 -0)
📝 app/Providers/Filament/ServerPanelProvider.php (+1 -0)
📝 app/Services/Subusers/SubuserCreationService.php (+2 -6)
📝 app/Services/Subusers/SubuserDeletionService.php (+2 -5)
📝 tests/Feature/Webhooks/ProcessWebhooksTest.php (+1 -1)

📄 Description

Database notifications are sent for the following events:

  • Server is (re-) installed
  • User is added as subuser
  • User is removed as subuser

(Note: the mail sending for subusers is now also handled via events)

image


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/pelican-dev/panel/pull/817 **Author:** [@Boy132](https://github.com/Boy132) **Created:** 12/11/2024 **Status:** ✅ Merged **Merged:** 12/12/2024 **Merged by:** [@Boy132](https://github.com/Boy132) **Base:** `main` ← **Head:** `feature/database-notifications` --- ### 📝 Commits (10+) - [`091d0b4`](https://github.com/pelican-dev/panel/commit/091d0b4f093990ab255adf197f1ad76fd57c92cf) add database notifications to all panels - [`e9ab70c`](https://github.com/pelican-dev/panel/commit/e9ab70ca048fb904710b7182f3a9a6cf7e59a46d) add successful param to Installed event - [`1358c01`](https://github.com/pelican-dev/panel/commit/1358c01aabf6c8ad1398a7fffdd213505c3d3d72) add listener for Installed event - [`6153bee`](https://github.com/pelican-dev/panel/commit/6153beeb3054266f9a8f3333ebcf94324bd03c64) create event for subuser creation - [`c1dde6e`](https://github.com/pelican-dev/panel/commit/c1dde6e338d285da3ca082df9398361d191b8f6c) add listener for SubUserAdded event - [`9cbeebc`](https://github.com/pelican-dev/panel/commit/9cbeebc922f873a02008bc9e4825d2fd788c9b77) always send Installed event - [`79eab82`](https://github.com/pelican-dev/panel/commit/79eab82f41c84550392d3640a5148186e06552e4) create event for subuser removal - [`75337ce`](https://github.com/pelican-dev/panel/commit/75337ce6e8ea572b69690547644dcb8bc588089a) add listener for SubUserRemoved event - [`f2d676b`](https://github.com/pelican-dev/panel/commit/f2d676b5a51c36d6b5b3c442a044ad1d72ee3be0) add prefix to server name - [`d4fa109`](https://github.com/pelican-dev/panel/commit/d4fa1097344f9cefac36ae04ad2130cb1acbdac2) remove view action from SubUserRemoved notification ### 📊 Changes **16 files changed** (+176 additions, -45 deletions) <details> <summary>View changed files</summary> 📝 `app/Events/Server/Installed.php` (+1 -1) ➕ `app/Events/Server/SubUserAdded.php` (+17 -0) ➕ `app/Events/Server/SubUserRemoved.php` (+18 -0) 📝 `app/Http/Controllers/Api/Remote/Servers/ServerInstallController.php` (+5 -15) ➕ `app/Listeners/Server/ServerInstalledListener.php` (+29 -0) ➕ `app/Listeners/Server/SubUserAddedListener.php` (+29 -0) ➕ `app/Listeners/Server/SubUserRemovedListener.php` (+17 -0) 📝 `app/Notifications/AddedToServer.php` (+18 -5) 📝 `app/Notifications/RemovedFromServer.php` (+18 -5) 📝 `app/Notifications/ServerInstalled.php` (+16 -7) 📝 `app/Providers/Filament/AdminPanelProvider.php` (+1 -0) 📝 `app/Providers/Filament/AppPanelProvider.php` (+1 -0) 📝 `app/Providers/Filament/ServerPanelProvider.php` (+1 -0) 📝 `app/Services/Subusers/SubuserCreationService.php` (+2 -6) 📝 `app/Services/Subusers/SubuserDeletionService.php` (+2 -5) 📝 `tests/Feature/Webhooks/ProcessWebhooksTest.php` (+1 -1) </details> ### 📄 Description Database notifications are sent for the following events: - Server is (re-) installed - User is added as subuser - User is removed as subuser _(Note: the mail sending for subusers is now also handled via events)_ ![image](https://github.com/user-attachments/assets/9a028f59-3612-4c7b-b56c-d75724a5b054) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2026-02-05 18:00:28 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/panel-pelican-dev#988