[PR #177] [MERGED] chore: code cleanup, remove unused code and reformat migrations #168

New issue

Closed

opened 2026-02-26 17:37:36 +03:00 by kerem · 0 comments

kerem commented

2026-02-26 17:37:36 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/Corsinvest/cv4pve-admin/pull/177
Author: @franklupo
Created: 2/18/2026
Status: ✅ Merged
Merged: 2/18/2026
Merged by: @franklupo

Base: main ← Head: chore/cleanup-and-reformat-migrations

📝 Commits (1)

20c40f8 chore: code cleanup, remove unused code and reformat migrations

📊 Changes

7 files changed (+80 additions, -95 deletions)

View changed files

📝 src/Corsinvest.ProxmoxVE.Admin.Core/Components/Layout/CommandPalette.razor.cs (+1 -1)
📝 src/Corsinvest.ProxmoxVE.Admin.Core/Components/ProxmoxVE/Cluster/Snapshots.razor.cs (+0 -2)
📝 src/Corsinvest.ProxmoxVE.Admin.Module.AutoSnap/Migrations/20260216135651_AddExtendedDataRemoveWebHooks.cs (+57 -58)
📝 src/Corsinvest.ProxmoxVE.Admin.Module.Bots/Telegram/Components/Render.razor.cs (+1 -2)
📝 src/Corsinvest.ProxmoxVE.Admin.Module.Dashboard/Migrations/20260217125654_RemoveWidgetModuleType.cs (+21 -22)
📝 src/Corsinvest.ProxmoxVE.Admin.Module.Updater/Components/Scans.razor (+0 -9)
📝 src/Corsinvest.ProxmoxVE.Admin.Module.Updater/Components/Scans.razor.cs (+0 -1)

📄 Description

Summary

CommandPalette: fix nullable Dictionary<string, object?>
Snapshots: remove unused AllowCalculateSnapshotSize property
Render (Bots/Telegram): remove unused DialogService dependency
Scans (Updater): remove commented-out Upgrade button and unused IsUpgrade property
Migrations (AddExtendedDataRemoveWebHooks, RemoveWidgetModuleType): reformat to file-scoped namespace style

Test plan

Verify app builds without warnings
Verify CommandPalette works correctly
Verify Scans page in Updater module renders correctly

_{🔄 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/Corsinvest/cv4pve-admin/pull/177 **Author:** [@franklupo](https://github.com/franklupo) **Created:** 2/18/2026 **Status:** ✅ Merged **Merged:** 2/18/2026 **Merged by:** [@franklupo](https://github.com/franklupo) **Base:** `main` ← **Head:** `chore/cleanup-and-reformat-migrations` --- ### 📝 Commits (1) - [`20c40f8`](https://github.com/Corsinvest/cv4pve-admin/commit/20c40f8076e9080a3e888c1d594ce804544db638) chore: code cleanup, remove unused code and reformat migrations ### 📊 Changes **7 files changed** (+80 additions, -95 deletions) <details> <summary>View changed files</summary> 📝 `src/Corsinvest.ProxmoxVE.Admin.Core/Components/Layout/CommandPalette.razor.cs` (+1 -1) 📝 `src/Corsinvest.ProxmoxVE.Admin.Core/Components/ProxmoxVE/Cluster/Snapshots.razor.cs` (+0 -2) 📝 `src/Corsinvest.ProxmoxVE.Admin.Module.AutoSnap/Migrations/20260216135651_AddExtendedDataRemoveWebHooks.cs` (+57 -58) 📝 `src/Corsinvest.ProxmoxVE.Admin.Module.Bots/Telegram/Components/Render.razor.cs` (+1 -2) 📝 `src/Corsinvest.ProxmoxVE.Admin.Module.Dashboard/Migrations/20260217125654_RemoveWidgetModuleType.cs` (+21 -22) 📝 `src/Corsinvest.ProxmoxVE.Admin.Module.Updater/Components/Scans.razor` (+0 -9) 📝 `src/Corsinvest.ProxmoxVE.Admin.Module.Updater/Components/Scans.razor.cs` (+0 -1) </details> ### 📄 Description ## Summary - `CommandPalette`: fix nullable `Dictionary<string, object?>` - `Snapshots`: remove unused `AllowCalculateSnapshotSize` property - `Render` (Bots/Telegram): remove unused `DialogService` dependency - `Scans` (Updater): remove commented-out Upgrade button and unused `IsUpgrade` property - Migrations (`AddExtendedDataRemoveWebHooks`, `RemoveWidgetModuleType`): reformat to file-scoped namespace style ## Test plan - [ ] Verify app builds without warnings - [ ] Verify CommandPalette works correctly - [ ] Verify Scans page in Updater module renders correctly --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>