[PR #701] [MERGED] Fix AutoDeploy & KeyCreationService #906

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

📋 Pull Request Information

Original PR: https://github.com/pelican-dev/panel/pull/701
Author: @rmartinoscar
Created: 11/7/2024
Status: Merged
Merged: 11/8/2024
Merged by: @notAreYouScared

Base: mainHead: fix/autodeploy-keys


📝 Commits (2)

  • d2488e8 Fix AutoDeploy & KeyCreationService
  • 33a227d Get rid of 2nd param & unset perm

📊 Changes

3 files changed (+7 additions, -5 deletions)

View changed files

📝 app/Http/Controllers/Admin/ApiController.php (+2 -1)
📝 app/Services/Api/KeyCreationService.php (+3 -3)
📝 app/Services/Nodes/NodeAutoDeployService.php (+2 -1)

📄 Description

Second param of handle() is a 1d array


🔄 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/701 **Author:** [@rmartinoscar](https://github.com/rmartinoscar) **Created:** 11/7/2024 **Status:** ✅ Merged **Merged:** 11/8/2024 **Merged by:** [@notAreYouScared](https://github.com/notAreYouScared) **Base:** `main` ← **Head:** `fix/autodeploy-keys` --- ### 📝 Commits (2) - [`d2488e8`](https://github.com/pelican-dev/panel/commit/d2488e80ebe87b4a23ce6e48b1996f27f754015c) Fix AutoDeploy & KeyCreationService - [`33a227d`](https://github.com/pelican-dev/panel/commit/33a227dd3173cf1101ccca3ca5aa56722407d23c) Get rid of 2nd param & unset perm ### 📊 Changes **3 files changed** (+7 additions, -5 deletions) <details> <summary>View changed files</summary> 📝 `app/Http/Controllers/Admin/ApiController.php` (+2 -1) 📝 `app/Services/Api/KeyCreationService.php` (+3 -3) 📝 `app/Services/Nodes/NodeAutoDeployService.php` (+2 -1) </details> ### 📄 Description Second param of handle() is a 1d array --- <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:01:02 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/panel#906