[PR #33] [MERGED] Add user server deletion capability #75

New issue

Closed

opened 2026-05-07 00:17:56 +02:00 by BreizhHardware · 0 comments

BreizhHardware commented

2026-05-07 00:17:56 +02:00

Owner

📋 Pull Request Information

Original PR: https://github.com/pelican-dev/plugins/pull/33
Author: @zfelyx
Created: 12/21/2025
Status: ✅ Merged
Merged: 12/23/2025
Merged by: @Boy132

Base: main ← Head: feature/user-creatable-servers-delete-own-servers

📝 Commits (10+)

c8bf424 Add user server deletion capability
9a047c5 Allow admins to delete any server
1a1ff7f Fix null user check in DeleteServerAction
ccf511c remove hardcoded language
9d3b020 Translate success notification messages for server deletion
be5f393 Merge branch 'feature/user-creatable-servers-delete-own-servers' of https://github.com/invalidfelix/plugins into feature/user-creatable-servers-delete-own-servers
8836999 Remove unused DeleteServerAction class.
04ab1c0 Remove redundant user check in canView method
1065e94 Update server delete permission check
51ceb4f Redirect to default panel URL after server creation

📊 Changes

6 files changed (+55 additions, -5 deletions)

View changed files

📝 user-creatable-servers/config/user-creatable-servers.php (+1 -0)
📝 user-creatable-servers/lang/en/strings.php (+6 -0)
📝 user-creatable-servers/plugin.json (+1 -1)
📝 user-creatable-servers/src/Filament/Server/Pages/ServerResourcePage.php (+37 -1)
📝 user-creatable-servers/src/Models/UserResourceLimits.php (+3 -3)
📝 user-creatable-servers/src/UserCreatableServersPlugin.php (+7 -0)

📄 Description

Introduces the ability for users to delete their own servers if permitted by configuration. Adds related configuration, language strings, UI actions, and updates resource limit logic for improved robustness. Also updates plugin settings to allow toggling this feature. #23

Summary by CodeRabbit

New Features
- Server deletion: users can delete servers when enabled by admin; includes confirmation dialog, warning, and success/error notifications.
- Admin setting to toggle user deletion (default: disabled).
Bug Fixes
- Resource limit calculations now never return negative values.
Documentation / Minor
- Minor metadata/formatting tidy-up.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

_{🔄 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/plugins/pull/33 **Author:** [@zfelyx](https://github.com/zfelyx) **Created:** 12/21/2025 **Status:** ✅ Merged **Merged:** 12/23/2025 **Merged by:** [@Boy132](https://github.com/Boy132) **Base:** `main` ← **Head:** `feature/user-creatable-servers-delete-own-servers` --- ### 📝 Commits (10+) - [`c8bf424`](https://github.com/pelican-dev/plugins/commit/c8bf4243330af3d7261fb9a5377fac5fc0fdf532) Add user server deletion capability - [`9a047c5`](https://github.com/pelican-dev/plugins/commit/9a047c5376bf6a2ef0cfd19edb4f4500ea2b509e) Allow admins to delete any server - [`1a1ff7f`](https://github.com/pelican-dev/plugins/commit/1a1ff7f60458989f5e2f5e2221bf93da18363362) Fix null user check in DeleteServerAction - [`ccf511c`](https://github.com/pelican-dev/plugins/commit/ccf511c34c31ec0b35a1dd15a80de014f8e76201) remove hardcoded language - [`9d3b020`](https://github.com/pelican-dev/plugins/commit/9d3b020dc4ebb237f4400c62c9b4a4a74765e71b) Translate success notification messages for server deletion - [`be5f393`](https://github.com/pelican-dev/plugins/commit/be5f393d137f7c406a9d0e5775af9148c92c506f) Merge branch 'feature/user-creatable-servers-delete-own-servers' of https://github.com/invalidfelix/plugins into feature/user-creatable-servers-delete-own-servers - [`8836999`](https://github.com/pelican-dev/plugins/commit/8836999e4f579206638dfbeefd8b7961bda126cc) Remove unused DeleteServerAction class. - [`04ab1c0`](https://github.com/pelican-dev/plugins/commit/04ab1c0c0c67478008ad64df56b9975283b67994) Remove redundant user check in canView method - [`1065e94`](https://github.com/pelican-dev/plugins/commit/1065e945fe1cf604866228124c03ad6b5d7ad4f3) Update server delete permission check - [`51ceb4f`](https://github.com/pelican-dev/plugins/commit/51ceb4fc07a51b2f45a8ed4b8ca2d57ec3d7c83f) Redirect to default panel URL after server creation ### 📊 Changes **6 files changed** (+55 additions, -5 deletions) <details> <summary>View changed files</summary> 📝 `user-creatable-servers/config/user-creatable-servers.php` (+1 -0) 📝 `user-creatable-servers/lang/en/strings.php` (+6 -0) 📝 `user-creatable-servers/plugin.json` (+1 -1) 📝 `user-creatable-servers/src/Filament/Server/Pages/ServerResourcePage.php` (+37 -1) 📝 `user-creatable-servers/src/Models/UserResourceLimits.php` (+3 -3) 📝 `user-creatable-servers/src/UserCreatableServersPlugin.php` (+7 -0) </details> ### 📄 Description Introduces the ability for users to delete their own servers if permitted by configuration. Adds related configuration, language strings, UI actions, and updates resource limit logic for improved robustness. Also updates plugin settings to allow toggling this feature. #23  ## Summary by CodeRabbit * **New Features** * Server deletion: users can delete servers when enabled by admin; includes confirmation dialog, warning, and success/error notifications. * Admin setting to toggle user deletion (default: disabled). * **Bug Fixes** * Resource limit calculations now never return negative values. * **Documentation / Minor** * Minor metadata/formatting tidy-up. <sub>✏️ Tip: You can customize this high-level summary in your review settings.</sub>  --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>