Drop-OSS/drop
1
0
Fork 0
mirror of https://github.com/Drop-OSS/drop.git synced 2026-07-25 12:25:31 -04:00
Code Issues 77 Packages Projects Releases 10 Wiki Activity

[PR #336] [MERGED] Make application and logo configurable #318

New Issue
Closed
opened 2026-02-17 17:06:38 -05:00 by yindo · 0 comments
yindo commented 2026-02-17 17:06:38 -05:00
Owner
Copy Link
Copy Source

📋 Pull Request Information

Original PR: https://github.com/Drop-OSS/drop/pull/336
Author: @Pacodastre
Created: 2/5/2026
Status: ✅ Merged
Merged: 2/5/2026
Merged by: @DecDuck

Base: develop ← Head: make-application-and-logo-configurable

📝 Commits (6)

  • f750e1b Adds settings for server name and logo
  • 718caa6 Implements ApplicationLogo and replaces site name based on settings
  • 1271779 Refactors component for changing the company logo
  • eca9958 Removes unused variable
  • 825288e Uses message instead of statusMessage
  • 75d7bf0 Replaces favicon with logo if set

📊 Changes

27 files changed (+454 additions, -104 deletions)

View changed files

➕ components/ApplicationLogo.vue (+12 -0)
📝 components/DropWordmark.vue (+11 -2)
📝 components/GameCarousel.vue (+3 -1)
➕ components/ImageUpload.vue (+27 -0)
📝 components/StoreView.vue (+3 -1)
📝 components/UserHeader.vue (+1 -1)
📝 error.vue (+1 -1)
📝 i18n/locales/en_us.json (+14 -2)
📝 layouts/admin.vue (+14 -2)
📝 layouts/default.vue (+3 -0)
📝 pages/admin/index.vue (+1 -2)
📝 pages/admin/metadata/companies/[id]/index.vue (+6 -12)
📝 pages/admin/settings.vue (+9 -2)
📝 pages/admin/settings/index.vue (+126 -60)
➕ pages/admin/settings/store/index.vue (+110 -0)
📝 pages/auth/mfa.vue (+1 -1)
📝 pages/auth/register.vue (+1 -1)
📝 pages/auth/signin.vue (+1 -2)
➕ prisma/migrations/20260126230112_general_settings/migration.sql (+15 -0)
📝 prisma/models/app.prisma (+2 -0)

...and 7 more files

📄 Description

Adds 2 new settings for the serverName and the logo.

I refactored the image upload component from the company edit page.

I didn't change too many strings containing "Drop", just a few. We can change more if we think we need more.

🔄 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/Drop-OSS/drop/pull/336 **Author:** [@Pacodastre](https://github.com/Pacodastre) **Created:** 2/5/2026 **Status:** ✅ Merged **Merged:** 2/5/2026 **Merged by:** [@DecDuck](https://github.com/DecDuck) **Base:** `develop` ← **Head:** `make-application-and-logo-configurable` --- ### 📝 Commits (6) - [`f750e1b`](https://github.com/Drop-OSS/drop/commit/f750e1b7822fd6b05714300230b7eba1052df3fe) Adds settings for server name and logo - [`718caa6`](https://github.com/Drop-OSS/drop/commit/718caa6e4a346e8d0892fa032ed39aa76338738b) Implements ApplicationLogo and replaces site name based on settings - [`1271779`](https://github.com/Drop-OSS/drop/commit/1271779fa7b9aeaaf57d0e689d9fbd30868750e9) Refactors component for changing the company logo - [`eca9958`](https://github.com/Drop-OSS/drop/commit/eca995835cf1091e0dd2ff7e75d3f697f48e873e) Removes unused variable - [`825288e`](https://github.com/Drop-OSS/drop/commit/825288ecb0fc5b91fdb6c9de0c9a544b9ceec822) Uses message instead of statusMessage - [`75d7bf0`](https://github.com/Drop-OSS/drop/commit/75d7bf01bde74108607c91ee157af9b42f28b2fa) Replaces favicon with logo if set ### 📊 Changes **27 files changed** (+454 additions, -104 deletions) <details> <summary>View changed files</summary> ➕ `components/ApplicationLogo.vue` (+12 -0) 📝 `components/DropWordmark.vue` (+11 -2) 📝 `components/GameCarousel.vue` (+3 -1) ➕ `components/ImageUpload.vue` (+27 -0) 📝 `components/StoreView.vue` (+3 -1) 📝 `components/UserHeader.vue` (+1 -1) 📝 `error.vue` (+1 -1) 📝 `i18n/locales/en_us.json` (+14 -2) 📝 `layouts/admin.vue` (+14 -2) 📝 `layouts/default.vue` (+3 -0) 📝 `pages/admin/index.vue` (+1 -2) 📝 `pages/admin/metadata/companies/[id]/index.vue` (+6 -12) 📝 `pages/admin/settings.vue` (+9 -2) 📝 `pages/admin/settings/index.vue` (+126 -60) ➕ `pages/admin/settings/store/index.vue` (+110 -0) 📝 `pages/auth/mfa.vue` (+1 -1) 📝 `pages/auth/register.vue` (+1 -1) 📝 `pages/auth/signin.vue` (+1 -2) ➕ `prisma/migrations/20260126230112_general_settings/migration.sql` (+15 -0) 📝 `prisma/models/app.prisma` (+2 -0) _...and 7 more files_ </details> ### 📄 Description Adds 2 new settings for the serverName and the logo. I refactored the image upload component from the company edit page. I didn't change too many strings containing "Drop", just a few. We can change more if we think we need more. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-17 17:06:38 -05:00
yindo closed this issue 2026-02-17 17:06:38 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
confirmed
duplicate
feature
feature
good first issue
needs-triage
pull-request
Mirrored from GitHub Pull Request
 qol
support
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
yindo
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Drop-OSS/drop#318
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?