[PR #130] [MERGED] Pacodastre show title description setting 51 #221

Closed
opened 2026-02-17 17:06:27 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/Drop-OSS/drop/pull/130
Author: @DecDuck
Created: 7/5/2025
Status: Merged
Merged: 7/5/2025
Merged by: @DecDuck

Base: developHead: Pacodastre-show-title-description-setting-51


📝 Commits (8)

  • d7d9561 #51 Adds settings page with showTitleDescriptionOnGamePanel
  • 7addb6c Removes console.log
  • abb5e21 Renames isHidden to system, adds missing system column on Game and fixes nitro plugin on fresh database
  • 1c93871 Implements a different way to handle the placeholder image
  • 283e92e Removes system column on Game
  • 4170b3d Groups settings keys together
  • 88b6de4 Removes unused i18n keys
  • ce75f2b fix: fix eslints and other small tweaks

📊 Changes

18 files changed (+413 additions, -158 deletions)

View changed files

📝 components/GameCarousel.vue (+9 -1)
📝 components/GamePanel.vue (+57 -17)
components/OptionWrapper.vue (+22 -0)
📝 eslint.config.mjs (+1 -1)
📝 i18n/locales/en_us.json (+17 -2)
📝 layouts/admin.vue (+1 -1)
📝 pages/admin/settings/index.vue (+114 -5)
prisma/migrations/20250621205541_add_show_title_description_on_game_panel/migration.sql (+2 -0)
📝 prisma/models/app.prisma (+2 -0)
📝 prisma/models/client.prisma (+13 -13)
📝 prisma/models/content.prisma (+117 -117)
public/game-panel-placeholder.png (+0 -0)
server/api/v1/admin/settings/dummy-data.get.ts (+11 -0)
server/api/v1/admin/settings/index.get.ts (+13 -0)
server/api/v1/admin/settings/index.patch.ts (+23 -0)
📝 server/internal/acls/descriptions.ts (+4 -0)
📝 server/internal/acls/index.ts (+4 -0)
📝 server/internal/library/index.ts (+3 -1)

📄 Description

Had to recreate the PR because I can't push to Paco's branch.

Other PR: #124


🔄 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/130 **Author:** [@DecDuck](https://github.com/DecDuck) **Created:** 7/5/2025 **Status:** ✅ Merged **Merged:** 7/5/2025 **Merged by:** [@DecDuck](https://github.com/DecDuck) **Base:** `develop` ← **Head:** `Pacodastre-show-title-description-setting-51` --- ### 📝 Commits (8) - [`d7d9561`](https://github.com/Drop-OSS/drop/commit/d7d956178886505fabb3cd6cc22a5ec285de4e5f) #51 Adds settings page with showTitleDescriptionOnGamePanel - [`7addb6c`](https://github.com/Drop-OSS/drop/commit/7addb6c2dec5761bfaeb2242d5ff4b527b23a9bb) Removes console.log - [`abb5e21`](https://github.com/Drop-OSS/drop/commit/abb5e21e807565de86247bfa0aee1b6e86c3f469) Renames isHidden to system, adds missing system column on Game and fixes nitro plugin on fresh database - [`1c93871`](https://github.com/Drop-OSS/drop/commit/1c93871c04e38512be15dbb0861cfe8082cbb149) Implements a different way to handle the placeholder image - [`283e92e`](https://github.com/Drop-OSS/drop/commit/283e92e18e360b11d65c04add4a93d3f82585342) Removes system column on Game - [`4170b3d`](https://github.com/Drop-OSS/drop/commit/4170b3d8a2151e231d567509ebf82d8563a75881) Groups settings keys together - [`88b6de4`](https://github.com/Drop-OSS/drop/commit/88b6de43ef60403dd488b78013d1904ce09df72a) Removes unused i18n keys - [`ce75f2b`](https://github.com/Drop-OSS/drop/commit/ce75f2b76b71a3abc2650ed69030cee43c45bea9) fix: fix eslints and other small tweaks ### 📊 Changes **18 files changed** (+413 additions, -158 deletions) <details> <summary>View changed files</summary> 📝 `components/GameCarousel.vue` (+9 -1) 📝 `components/GamePanel.vue` (+57 -17) ➕ `components/OptionWrapper.vue` (+22 -0) 📝 `eslint.config.mjs` (+1 -1) 📝 `i18n/locales/en_us.json` (+17 -2) 📝 `layouts/admin.vue` (+1 -1) 📝 `pages/admin/settings/index.vue` (+114 -5) ➕ `prisma/migrations/20250621205541_add_show_title_description_on_game_panel/migration.sql` (+2 -0) 📝 `prisma/models/app.prisma` (+2 -0) 📝 `prisma/models/client.prisma` (+13 -13) 📝 `prisma/models/content.prisma` (+117 -117) ➕ `public/game-panel-placeholder.png` (+0 -0) ➕ `server/api/v1/admin/settings/dummy-data.get.ts` (+11 -0) ➕ `server/api/v1/admin/settings/index.get.ts` (+13 -0) ➕ `server/api/v1/admin/settings/index.patch.ts` (+23 -0) 📝 `server/internal/acls/descriptions.ts` (+4 -0) 📝 `server/internal/acls/index.ts` (+4 -0) 📝 `server/internal/library/index.ts` (+3 -1) </details> ### 📄 Description Had to recreate the PR because I can't push to Paco's branch. Other PR: #124 --- <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:27 -05:00
yindo closed this issue 2026-02-17 17:06:27 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Drop-OSS/drop#221