starred/panel-pelican-dev
1
0
Fork 0
mirror of https://github.com/pelican-dev/panel.git synced 2026-05-04 18:00:48 +03:00
Code Issues 41 Packages Projects Releases 10 Wiki Activity

[PR #1555] [MERGED] Refactor AlertBanner to be ViewComponent #1452

New Issue
Closed
opened 2026-02-05 18:14:28 +03:00 by OVERLORD · 0 comments
OVERLORD commented 2026-02-05 18:14:28 +03:00
Owner
Copy Link

📋 Pull Request Information

Original PR: https://github.com/pelican-dev/panel/pull/1555
Author: @Boy132
Created: 7/23/2025
Status: ✅ Merged
Merged: 8/1/2025
Merged by: @Boy132

Base: main ← Head: boy132/convert-alert-banner-to-view-component

📝 Commits (6)

  • 4e469ed convert AlertBanner to ViewComponent
  • e626ae4 small cleanup
  • 2469cc8 Merge remote-tracking branch 'upstream/main' into boy132/convert-alert-banner-to-view-component
  • 7a3f0ca make id required
  • b4708fa remove useless node check
  • e1de548 Merge remote-tracking branch 'upstream/main' into boy132/convert-alert-banner-to-view-component

📊 Changes

7 files changed (+50 additions, -50 deletions)

View changed files

📝 app/Filament/Server/Resources/FileResource/Pages/EditFiles.php (+3 -12)
📝 app/Filament/Server/Resources/FileResource/Pages/ListFiles.php (+2 -2)
📝 app/Livewire/AlertBanner.php (+27 -17)
📝 app/Livewire/AlertBannerContainer.php (+8 -7)
📝 app/Models/File.php (+1 -1)
📝 resources/views/livewire/alerts/alert-banner-container.blade.php (+2 -2)
📝 resources/views/livewire/alerts/alert-banner.blade.php (+7 -9)

📄 Description

This does not fix the need for a page refresh to show the alert banner.

🔄 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/1555 **Author:** [@Boy132](https://github.com/Boy132) **Created:** 7/23/2025 **Status:** ✅ Merged **Merged:** 8/1/2025 **Merged by:** [@Boy132](https://github.com/Boy132) **Base:** `main` ← **Head:** `boy132/convert-alert-banner-to-view-component` --- ### 📝 Commits (6) - [`4e469ed`](https://github.com/pelican-dev/panel/commit/4e469eda41d23d012ae4e650684e810322dc8a47) convert `AlertBanner` to `ViewComponent` - [`e626ae4`](https://github.com/pelican-dev/panel/commit/e626ae40e71f05e6a4679a029c8db58b1f9a79aa) small cleanup - [`2469cc8`](https://github.com/pelican-dev/panel/commit/2469cc80664f8263eb43096eb452f78522dab0e3) Merge remote-tracking branch 'upstream/main' into boy132/convert-alert-banner-to-view-component - [`7a3f0ca`](https://github.com/pelican-dev/panel/commit/7a3f0ca3e93581a6079a01d4b1d294255ac303e7) make id required - [`b4708fa`](https://github.com/pelican-dev/panel/commit/b4708fa6e3003f790bff4d764a484e96c46597b7) remove useless node check - [`e1de548`](https://github.com/pelican-dev/panel/commit/e1de5481cafb6f8b7ec26275f6ffe99068bdf4b7) Merge remote-tracking branch 'upstream/main' into boy132/convert-alert-banner-to-view-component ### 📊 Changes **7 files changed** (+50 additions, -50 deletions) <details> <summary>View changed files</summary> 📝 `app/Filament/Server/Resources/FileResource/Pages/EditFiles.php` (+3 -12) 📝 `app/Filament/Server/Resources/FileResource/Pages/ListFiles.php` (+2 -2) 📝 `app/Livewire/AlertBanner.php` (+27 -17) 📝 `app/Livewire/AlertBannerContainer.php` (+8 -7) 📝 `app/Models/File.php` (+1 -1) 📝 `resources/views/livewire/alerts/alert-banner-container.blade.php` (+2 -2) 📝 `resources/views/livewire/alerts/alert-banner.blade.php` (+7 -9) </details> ### 📄 Description _This does not fix the need for a page refresh to show the alert banner._ --- <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:14:28 +03:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
not confirmed
pull-request
Mirrored from GitHub Pull Request
 👑 sponsor
💰 fund
🔴 hard
🟡 medium
🟡 medium
🟡 medium
🟢 easy
🟢 easy
🟢 easy
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
OVERLORD
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/panel-pelican-dev#1452
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?