[PR #1290] [MERGED] Remove DynamicDatabaseConnection #1304

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

📋 Pull Request Information

Original PR: https://github.com/pelican-dev/panel/pull/1290
Author: @Boy132
Created: 4/22/2025
Status: Merged
Merged: 4/23/2025
Merged by: @Boy132

Base: mainHead: boy132/remove-DynamicDatabaseConnection


📝 Commits (1)

  • c4d4da4 remove DynamicDatabaseConnection

📊 Changes

7 files changed (+23 additions, -61 deletions)

View changed files

app/Extensions/DynamicDatabaseConnection.php (+0 -35)
📝 app/Models/Database.php (+1 -4)
📝 app/Models/DatabaseHost.php (+19 -0)
📝 app/Services/Databases/DatabaseManagementService.php (+0 -6)
📝 app/Services/Databases/DatabasePasswordService.php (+0 -4)
📝 app/Services/Databases/Hosts/HostCreationService.php (+1 -6)
📝 app/Services/Databases/Hosts/HostUpdateService.php (+2 -6)

📄 Description

Replaces our custom class by DB::build.


🔄 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/1290 **Author:** [@Boy132](https://github.com/Boy132) **Created:** 4/22/2025 **Status:** ✅ Merged **Merged:** 4/23/2025 **Merged by:** [@Boy132](https://github.com/Boy132) **Base:** `main` ← **Head:** `boy132/remove-DynamicDatabaseConnection` --- ### 📝 Commits (1) - [`c4d4da4`](https://github.com/pelican-dev/panel/commit/c4d4da42ff239fc424b8dc8e6055a6ed9c9716da) remove `DynamicDatabaseConnection` ### 📊 Changes **7 files changed** (+23 additions, -61 deletions) <details> <summary>View changed files</summary> ➖ `app/Extensions/DynamicDatabaseConnection.php` (+0 -35) 📝 `app/Models/Database.php` (+1 -4) 📝 `app/Models/DatabaseHost.php` (+19 -0) 📝 `app/Services/Databases/DatabaseManagementService.php` (+0 -6) 📝 `app/Services/Databases/DatabasePasswordService.php` (+0 -4) 📝 `app/Services/Databases/Hosts/HostCreationService.php` (+1 -6) 📝 `app/Services/Databases/Hosts/HostUpdateService.php` (+2 -6) </details> ### 📄 Description Replaces our custom class by `DB::build`. --- <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:35:10 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/panel#1304