From 4168f43e3ecd51479bec3aae585abbe6dcd3e416 Mon Sep 17 00:00:00 2001 From: Pujit Mehrotra Date: Wed, 9 Jul 2025 10:59:10 -0400 Subject: [PATCH] fix: duplicated header logo after api stops (#1493) Move legacy header logo omission from API file modifier to plg install step to avoid breaking rollback (ie upon `unraid-api stop`) due to web component upgrade. Tested on 7.1.4 & 7.2.0-beta.0.16 Testing & Reproduction procedure: 1. Install connect plugin 2. Run `unraid-api stop` on server 3. Refresh page. Before Unraid 7.2, Plugin versions prior to this will display a duplicated logo that blocks the nav menu. Now, it will not. Plugin uninstall behavior remains unchanged. ## Summary by CodeRabbit * **New Features** * The plugin installation process now updates the header logo by removing the old logo from the interface during installation. --- ...efaultPageLayout.php.modified.snapshot.php | 2 +- .../default-page-layout.modification.ts | 8 ------- .../patches/default-page-layout.patch | 10 ++------ plugin/plugins/dynamix.unraid.net.plg | 23 +++++++++++++++++++ 4 files changed, 26 insertions(+), 17 deletions(-) diff --git a/api/src/unraid-api/unraid-file-modifier/modifications/__test__/snapshots/DefaultPageLayout.php.modified.snapshot.php b/api/src/unraid-api/unraid-file-modifier/modifications/__test__/snapshots/DefaultPageLayout.php.modified.snapshot.php index c6ba6a642..62950004f 100644 --- a/api/src/unraid-api/unraid-file-modifier/modifications/__test__/snapshots/DefaultPageLayout.php.modified.snapshot.php +++ b/api/src/unraid-api/unraid-file-modifier/modifications/__test__/snapshots/DefaultPageLayout.php.modified.snapshot.php @@ -711,7 +711,7 @@ $.ajaxPrefilter(function(s, orig, xhr){