[PR #1327] [MERGED] Refactor Providers to be a singleton #1323

Closed
opened 2026-02-04 19:36:41 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/pelican-dev/panel/pull/1327
Author: @pelican-vehikl
Created: 5/1/2025
Status: Merged
Merged: 7/2/2025
Merged by: @lancepioch

Base: mainHead: vehikl/singleton


📝 Commits (10+)

  • fa5fee5 Refactor OauthProviders to use Singleton
  • 3053741 Add OAuthServiceProvider
  • e349f3f Clean up constructors
  • 4334524 WIP not working Feature Singleton
  • 6818ee8 Merge branch 'main' into vehikl/singleton
  • 55fd291 Merge branch 'main' into vehikl/singleton
  • 8ee0005 Features
  • 1dc1b13 Merge branch 'main' into vehikl/singleton
  • d9257fd Avatars
  • 09e3506 Merge branch 'main' into vehikl/singleton

📊 Changes

45 files changed (+643 additions, -567 deletions)

View changed files

app/Extensions/Avatar/AvatarProvider.php (+0 -42)
app/Extensions/Avatar/AvatarSchemaInterface.php (+14 -0)
app/Extensions/Avatar/AvatarService.php (+55 -0)
📝 app/Extensions/Avatar/Schemas/GravatarSchema.php (+7 -7)
📝 app/Extensions/Avatar/Schemas/UiAvatarsSchema.php (+3 -8)
app/Extensions/Captcha/CaptchaService.php (+48 -0)
📝 app/Extensions/Captcha/Schemas/BaseSchema.php (+5 -59)
app/Extensions/Captcha/Schemas/CaptchaSchemaInterface.php (+35 -0)
📝 app/Extensions/Captcha/Schemas/Turnstile/Component.php (+3 -6)
📝 app/Extensions/Captcha/Schemas/Turnstile/Rule.php (+5 -4)
📝 app/Extensions/Captcha/Schemas/Turnstile/TurnstileSchema.php (+14 -15)
app/Extensions/Features/FeatureProvider.php (+0 -51)
app/Extensions/Features/FeatureSchemaInterface.php (+15 -0)
app/Extensions/Features/FeatureService.php (+52 -0)
📝 app/Extensions/Features/Schemas/GSLTokenSchema.php (+3 -13)
📝 app/Extensions/Features/Schemas/JavaVersionSchema.php (+3 -13)
📝 app/Extensions/Features/Schemas/MinecraftEulaSchema.php (+3 -13)
📝 app/Extensions/Features/Schemas/PIDLimitSchema.php (+3 -13)
📝 app/Extensions/Features/Schemas/SteamDiskSpaceSchema.php (+3 -13)
app/Extensions/OAuth/OAuthSchemaInterface.php (+35 -0)

...and 25 more files

📄 Description

  • Oauth
  • Captcha
  • Avatar
  • EggFeature

🔄 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/1327 **Author:** [@pelican-vehikl](https://github.com/pelican-vehikl) **Created:** 5/1/2025 **Status:** ✅ Merged **Merged:** 7/2/2025 **Merged by:** [@lancepioch](https://github.com/lancepioch) **Base:** `main` ← **Head:** `vehikl/singleton` --- ### 📝 Commits (10+) - [`fa5fee5`](https://github.com/pelican-dev/panel/commit/fa5fee50a0f6c1edf1544ff7f8b7d38fcddc9d07) Refactor OauthProviders to use Singleton - [`3053741`](https://github.com/pelican-dev/panel/commit/305374192fe9064d7abfdc4e5373d1b80020c5ed) Add `OAuthServiceProvider` - [`e349f3f`](https://github.com/pelican-dev/panel/commit/e349f3fc5896789c01da666668affab4aa70438f) Clean up constructors - [`4334524`](https://github.com/pelican-dev/panel/commit/43345240c52a44beb6cd364139d7e0d13c151b45) WIP not working Feature Singleton - [`6818ee8`](https://github.com/pelican-dev/panel/commit/6818ee8ee6c582f12a8f82fe19b4d20c6644542f) Merge branch 'main' into vehikl/singleton - [`55fd291`](https://github.com/pelican-dev/panel/commit/55fd291dcc331e70b5ad080417ef150b0da18113) Merge branch 'main' into vehikl/singleton - [`8ee0005`](https://github.com/pelican-dev/panel/commit/8ee00050c9c339578a0767f76343001e81bbd318) Features - [`1dc1b13`](https://github.com/pelican-dev/panel/commit/1dc1b13e8e705c63620983c96fc317592bc4db3f) Merge branch 'main' into vehikl/singleton - [`d9257fd`](https://github.com/pelican-dev/panel/commit/d9257fdb4f9672525efbde3b10762fc5a14732fa) Avatars - [`09e3506`](https://github.com/pelican-dev/panel/commit/09e3506efa47c4da097f94dcc8cc9aaeca370fc0) Merge branch 'main' into vehikl/singleton ### 📊 Changes **45 files changed** (+643 additions, -567 deletions) <details> <summary>View changed files</summary> ➖ `app/Extensions/Avatar/AvatarProvider.php` (+0 -42) ➕ `app/Extensions/Avatar/AvatarSchemaInterface.php` (+14 -0) ➕ `app/Extensions/Avatar/AvatarService.php` (+55 -0) 📝 `app/Extensions/Avatar/Schemas/GravatarSchema.php` (+7 -7) 📝 `app/Extensions/Avatar/Schemas/UiAvatarsSchema.php` (+3 -8) ➕ `app/Extensions/Captcha/CaptchaService.php` (+48 -0) 📝 `app/Extensions/Captcha/Schemas/BaseSchema.php` (+5 -59) ➕ `app/Extensions/Captcha/Schemas/CaptchaSchemaInterface.php` (+35 -0) 📝 `app/Extensions/Captcha/Schemas/Turnstile/Component.php` (+3 -6) 📝 `app/Extensions/Captcha/Schemas/Turnstile/Rule.php` (+5 -4) 📝 `app/Extensions/Captcha/Schemas/Turnstile/TurnstileSchema.php` (+14 -15) ➖ `app/Extensions/Features/FeatureProvider.php` (+0 -51) ➕ `app/Extensions/Features/FeatureSchemaInterface.php` (+15 -0) ➕ `app/Extensions/Features/FeatureService.php` (+52 -0) 📝 `app/Extensions/Features/Schemas/GSLTokenSchema.php` (+3 -13) 📝 `app/Extensions/Features/Schemas/JavaVersionSchema.php` (+3 -13) 📝 `app/Extensions/Features/Schemas/MinecraftEulaSchema.php` (+3 -13) 📝 `app/Extensions/Features/Schemas/PIDLimitSchema.php` (+3 -13) 📝 `app/Extensions/Features/Schemas/SteamDiskSpaceSchema.php` (+3 -13) ➕ `app/Extensions/OAuth/OAuthSchemaInterface.php` (+35 -0) _...and 25 more files_ </details> ### 📄 Description - [X] Oauth - [x] Captcha - [x] Avatar - [x] EggFeature --- <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 19:36:41 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/panel#1323