[PR #149] [MERGED] chore: unify branding to single logo and remove unused files #142

New issue

Closed

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

kerem commented

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

Owner

📋 Pull Request Information

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

Base: main ← Head: chore/unify-branding

📝 Commits (1)

93016e9 chore: unify branding to single logo and remove unused files

📊 Changes

16 files changed (+5 additions, -70 deletions)

View changed files

📝 CODE_OF_CONDUCT.md (+1 -1)
📝 src/Corsinvest.ProxmoxVE.Admin.Core/ServiceCollectionExtensions.cs (+1 -1)
➖ src/Corsinvest.ProxmoxVE.Admin.Core/Services/PublishMode.cs (+0 -35)
📝 src/Corsinvest.ProxmoxVE.Admin.Module.System/Security/Components/Account/Layouts/LoginLayout.razor (+1 -1)
📝 src/Corsinvest.ProxmoxVE.Admin/Components/App.razor (+2 -7)
📝 src/Corsinvest.ProxmoxVE.Admin/Components/Pages/Home.razor (+0 -6)
➖ src/Corsinvest.ProxmoxVE.Admin/wwwroot/android-chrome-192x192.png (+0 -0)
➖ src/Corsinvest.ProxmoxVE.Admin/wwwroot/android-chrome-512x512.png (+0 -0)
➖ src/Corsinvest.ProxmoxVE.Admin/wwwroot/apple-touch-icon.png (+0 -0)
➖ src/Corsinvest.ProxmoxVE.Admin/wwwroot/favicon-16x16.png (+0 -0)
➖ src/Corsinvest.ProxmoxVE.Admin/wwwroot/favicon-32x32.png (+0 -0)
➖ src/Corsinvest.ProxmoxVE.Admin/wwwroot/favicon.ico (+0 -0)
➖ src/Corsinvest.ProxmoxVE.Admin/wwwroot/images/cv4pve.png (+0 -0)
➖ src/Corsinvest.ProxmoxVE.Admin/wwwroot/images/logo-login.png (+0 -0)
📝 src/Corsinvest.ProxmoxVE.Admin/wwwroot/images/logo.png (+0 -0)
➖ src/Corsinvest.ProxmoxVE.Admin/wwwroot/site.webmanifest (+0 -19)

📄 Description

Summary

Replace multiple favicon/icon files with single images/logo.png
Remove site.webmanifest, legacy android/apple icons and old favicon files
Update App.razor and LoginLayout.razor to use unified logo.png
Fix typo 'Corisnvest' → 'Corsinvest' in CODE_OF_CONDUCT.md and App.razor
Remove unused PublishMode.cs (file contained only dead/commented-out code)
Sort using directives alphabetically in ServiceCollectionExtensions.cs
Remove commented-out RadzenStack block in Home.razor

Test plan

App loads and favicon shows correctly in browser tab
Login page shows correct logo
Build passes (no missing file references)

_{🔄 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/149 **Author:** [@franklupo](https://github.com/franklupo) **Created:** 2/16/2026 **Status:** ✅ Merged **Merged:** 2/16/2026 **Merged by:** [@franklupo](https://github.com/franklupo) **Base:** `main` ← **Head:** `chore/unify-branding` --- ### 📝 Commits (1) - [`93016e9`](https://github.com/Corsinvest/cv4pve-admin/commit/93016e95d99dd59cb1e8278e1f566e602a368f70) chore: unify branding to single logo and remove unused files ### 📊 Changes **16 files changed** (+5 additions, -70 deletions) <details> <summary>View changed files</summary> 📝 `CODE_OF_CONDUCT.md` (+1 -1) 📝 `src/Corsinvest.ProxmoxVE.Admin.Core/ServiceCollectionExtensions.cs` (+1 -1) ➖ `src/Corsinvest.ProxmoxVE.Admin.Core/Services/PublishMode.cs` (+0 -35) 📝 `src/Corsinvest.ProxmoxVE.Admin.Module.System/Security/Components/Account/Layouts/LoginLayout.razor` (+1 -1) 📝 `src/Corsinvest.ProxmoxVE.Admin/Components/App.razor` (+2 -7) 📝 `src/Corsinvest.ProxmoxVE.Admin/Components/Pages/Home.razor` (+0 -6) ➖ `src/Corsinvest.ProxmoxVE.Admin/wwwroot/android-chrome-192x192.png` (+0 -0) ➖ `src/Corsinvest.ProxmoxVE.Admin/wwwroot/android-chrome-512x512.png` (+0 -0) ➖ `src/Corsinvest.ProxmoxVE.Admin/wwwroot/apple-touch-icon.png` (+0 -0) ➖ `src/Corsinvest.ProxmoxVE.Admin/wwwroot/favicon-16x16.png` (+0 -0) ➖ `src/Corsinvest.ProxmoxVE.Admin/wwwroot/favicon-32x32.png` (+0 -0) ➖ `src/Corsinvest.ProxmoxVE.Admin/wwwroot/favicon.ico` (+0 -0) ➖ `src/Corsinvest.ProxmoxVE.Admin/wwwroot/images/cv4pve.png` (+0 -0) ➖ `src/Corsinvest.ProxmoxVE.Admin/wwwroot/images/logo-login.png` (+0 -0) 📝 `src/Corsinvest.ProxmoxVE.Admin/wwwroot/images/logo.png` (+0 -0) ➖ `src/Corsinvest.ProxmoxVE.Admin/wwwroot/site.webmanifest` (+0 -19) </details> ### 📄 Description ## Summary - Replace multiple favicon/icon files with single `images/logo.png` - Remove `site.webmanifest`, legacy android/apple icons and old favicon files - Update `App.razor` and `LoginLayout.razor` to use unified `logo.png` - Fix typo 'Corisnvest' → 'Corsinvest' in `CODE_OF_CONDUCT.md` and `App.razor` - Remove unused `PublishMode.cs` (file contained only dead/commented-out code) - Sort `using` directives alphabetically in `ServiceCollectionExtensions.cs` - Remove commented-out `RadzenStack` block in `Home.razor` ## Test plan - [ ] App loads and favicon shows correctly in browser tab - [ ] Login page shows correct logo - [ ] Build passes (no missing file references) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>