[PR #1195] [MERGED] Use afterSave instead of handleRecordUpdate & move transferServer #1238

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

📋 Pull Request Information

Original PR: https://github.com/pelican-dev/panel/pull/1195
Author: @rmartinoscar
Created: 4/1/2025
Status: Merged
Merged: 4/3/2025
Merged by: @rmartinoscar

Base: mainHead: fix/EditNode-EditServer


📝 Commits (2)

  • 7ceeb38 Use afterSave instead of handleRecordUpdate & move transferServer
  • 1c481ac Override getSavedNotification instead of save

📊 Changes

2 files changed (+71 additions, -93 deletions)

View changed files

📝 app/Filament/Admin/Resources/NodeResource/Pages/EditNode.php (+30 -36)
📝 app/Filament/Admin/Resources/ServerResource/Pages/EditServer.php (+41 -57)

📄 Description

Skip wings sync if field isn't relevant.


🔄 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/1195 **Author:** [@rmartinoscar](https://github.com/rmartinoscar) **Created:** 4/1/2025 **Status:** ✅ Merged **Merged:** 4/3/2025 **Merged by:** [@rmartinoscar](https://github.com/rmartinoscar) **Base:** `main` ← **Head:** `fix/EditNode-EditServer` --- ### 📝 Commits (2) - [`7ceeb38`](https://github.com/pelican-dev/panel/commit/7ceeb386c8f1f42313b1591d6b0004146ba79ca1) Use `afterSave` instead of `handleRecordUpdate` & move `transferServer` - [`1c481ac`](https://github.com/pelican-dev/panel/commit/1c481acc66e5a6c6edad342520da70133a3455e5) Override `getSavedNotification` instead of `save` ### 📊 Changes **2 files changed** (+71 additions, -93 deletions) <details> <summary>View changed files</summary> 📝 `app/Filament/Admin/Resources/NodeResource/Pages/EditNode.php` (+30 -36) 📝 `app/Filament/Admin/Resources/ServerResource/Pages/EditServer.php` (+41 -57) </details> ### 📄 Description Skip wings sync if field isn't relevant. --- <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:29:18 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/panel#1238