[PR #2090] [MERGED] Fix allocation policy for admins and update checks #1754

Closed
opened 2026-02-04 20:13:00 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/pelican-dev/panel/pull/2090
Author: @Boy132
Created: 1/9/2026
Status: Merged
Merged: 1/19/2026
Merged by: @Boy132

Base: mainHead: boy132/fix-allocation-policy


📝 Commits (5)

  • 68d468e fix allocation policy for admins
  • 1bfe827 fix update check and rename model param
  • 8a686d6 fix typo
  • 1548346 add model name
  • 4ac964d Merge branch 'main' into boy132/fix-allocation-policy

📊 Changes

6 files changed (+57 additions, -22 deletions)

View changed files

📝 app/Policies/AllocationPolicy.php (+43 -8)
📝 app/Policies/BackupPolicy.php (+2 -2)
📝 app/Policies/DatabasePolicy.php (+3 -3)
📝 app/Policies/FilePolicy.php (+3 -3)
📝 app/Policies/SchedulePolicy.php (+3 -3)
📝 app/Policies/SubuserPolicy.php (+3 -3)

📄 Description

The AllocationPolicy is used both in the admin area and the client area but only checked for subuser permissions. Now it will check for admin permissions if not tenant is set. (aka when you are not in the client area)

Also fixes the method name for the update checks.


🔄 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/2090 **Author:** [@Boy132](https://github.com/Boy132) **Created:** 1/9/2026 **Status:** ✅ Merged **Merged:** 1/19/2026 **Merged by:** [@Boy132](https://github.com/Boy132) **Base:** `main` ← **Head:** `boy132/fix-allocation-policy` --- ### 📝 Commits (5) - [`68d468e`](https://github.com/pelican-dev/panel/commit/68d468e59e586bbf7f6b193c5c431f0a64f6dd69) fix allocation policy for admins - [`1bfe827`](https://github.com/pelican-dev/panel/commit/1bfe827331e4ffca99843165d0136853218d107a) fix update check and rename model param - [`8a686d6`](https://github.com/pelican-dev/panel/commit/8a686d68a24c1f84713ddb12f639110bb9efd8e4) fix typo - [`1548346`](https://github.com/pelican-dev/panel/commit/154834676008f5a9f408372dba584a27f2f84ce9) add model name - [`4ac964d`](https://github.com/pelican-dev/panel/commit/4ac964d2086e36790fff05abfe3375941e486ee6) Merge branch 'main' into boy132/fix-allocation-policy ### 📊 Changes **6 files changed** (+57 additions, -22 deletions) <details> <summary>View changed files</summary> 📝 `app/Policies/AllocationPolicy.php` (+43 -8) 📝 `app/Policies/BackupPolicy.php` (+2 -2) 📝 `app/Policies/DatabasePolicy.php` (+3 -3) 📝 `app/Policies/FilePolicy.php` (+3 -3) 📝 `app/Policies/SchedulePolicy.php` (+3 -3) 📝 `app/Policies/SubuserPolicy.php` (+3 -3) </details> ### 📄 Description The `AllocationPolicy` is used both in the admin area and the client area but only checked for subuser permissions. Now it will check for admin permissions if not tenant is set. (aka when you are not in the client area) Also fixes the method name for the update checks. --- <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-04 20:13:00 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/panel#1754