[PR #2142] [MERGED] Fix oauth provider "enabled" checks #1781

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

📋 Pull Request Information

Original PR: https://github.com/pelican-dev/panel/pull/2142
Author: @Boy132
Created: 1/26/2026
Status: Merged
Merged: 1/27/2026
Merged by: @Boy132

Base: mainHead: boy132/fix-oauth-enabled-check


📝 Commits (1)

  • 84d5928 fix oauth provider "enabled" checks

📊 Changes

2 files changed (+5 additions, -8 deletions)

View changed files

📝 app/Extensions/OAuth/Schemas/OAuthSchema.php (+1 -3)
📝 app/Filament/Admin/Pages/Settings.php (+4 -5)

📄 Description

Use getConfigKey in the default isEnabled and call isEnabled instead of manually checking the config key.


🔄 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/2142 **Author:** [@Boy132](https://github.com/Boy132) **Created:** 1/26/2026 **Status:** ✅ Merged **Merged:** 1/27/2026 **Merged by:** [@Boy132](https://github.com/Boy132) **Base:** `main` ← **Head:** `boy132/fix-oauth-enabled-check` --- ### 📝 Commits (1) - [`84d5928`](https://github.com/pelican-dev/panel/commit/84d5928e10b0dd9bbc84a02b24fffafedc7b6e8a) fix oauth provider "enabled" checks ### 📊 Changes **2 files changed** (+5 additions, -8 deletions) <details> <summary>View changed files</summary> 📝 `app/Extensions/OAuth/Schemas/OAuthSchema.php` (+1 -3) 📝 `app/Filament/Admin/Pages/Settings.php` (+4 -5) </details> ### 📄 Description Use `getConfigKey` in the default `isEnabled` and call `isEnabled` instead of manually checking the config key. --- <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:24:29 +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#1781