[PR #1806] [MERGED] feat: rework playlist cover upload/remove #1880

Closed
opened 2026-02-26 03:32:39 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/koel/koel/pull/1806
Author: @phanan
Created: 7/26/2024
Status: Merged
Merged: 7/26/2024
Merged by: @phanan

Base: masterHead: playlist-cover


📝 Commits (1)

  • e39620c feat: rework playlist cover upload/remove

📊 Changes

12 files changed (+147 additions, -107 deletions)

View changed files

📝 .github/workflows/unit-frontend.yml (+1 -1)
app/Http/Controllers/API/PlaylistCoverController.php (+31 -0)
app/Http/Controllers/API/UploadPlaylistCoverController.php (+0 -22)
📝 app/Http/Requests/API/PlaylistCoverUpdateRequest.php (+1 -1)
📝 app/Services/MediaMetadataService.php (+8 -0)
📝 resources/assets/js/components/playlist/PlaylistContextMenu.vue (+4 -2)
📝 resources/assets/js/components/screens/PlaylistScreen.spec.ts (+2 -0)
📝 resources/assets/js/components/ui/PlaylistThumbnail.vue (+52 -59)
📝 resources/assets/js/stores/playlistStore.ts (+5 -0)
📝 routes/api.base.php (+3 -2)
📝 tests/Feature/KoelPlus/PlaylistCoverTest.php (+11 -0)
📝 tests/Feature/PlaylistCoverTest.php (+29 -20)

📄 Description

  • Replaces the awkward "drag and drop to upload playlist cover" functionality with a normal button
  • Adds the ability to remove a playlist's cover

🔄 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/koel/koel/pull/1806 **Author:** [@phanan](https://github.com/phanan) **Created:** 7/26/2024 **Status:** ✅ Merged **Merged:** 7/26/2024 **Merged by:** [@phanan](https://github.com/phanan) **Base:** `master` ← **Head:** `playlist-cover` --- ### 📝 Commits (1) - [`e39620c`](https://github.com/koel/koel/commit/e39620c1f5ca4c10c5f2b38ae6e2eaa8b5c98d12) feat: rework playlist cover upload/remove ### 📊 Changes **12 files changed** (+147 additions, -107 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/unit-frontend.yml` (+1 -1) ➕ `app/Http/Controllers/API/PlaylistCoverController.php` (+31 -0) ➖ `app/Http/Controllers/API/UploadPlaylistCoverController.php` (+0 -22) 📝 `app/Http/Requests/API/PlaylistCoverUpdateRequest.php` (+1 -1) 📝 `app/Services/MediaMetadataService.php` (+8 -0) 📝 `resources/assets/js/components/playlist/PlaylistContextMenu.vue` (+4 -2) 📝 `resources/assets/js/components/screens/PlaylistScreen.spec.ts` (+2 -0) 📝 `resources/assets/js/components/ui/PlaylistThumbnail.vue` (+52 -59) 📝 `resources/assets/js/stores/playlistStore.ts` (+5 -0) 📝 `routes/api.base.php` (+3 -2) 📝 `tests/Feature/KoelPlus/PlaylistCoverTest.php` (+11 -0) 📝 `tests/Feature/PlaylistCoverTest.php` (+29 -20) </details> ### 📄 Description * Replaces the awkward "drag and drop to upload playlist cover" functionality with a normal button * Adds the ability to remove a playlist's cover --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-26 03:32:39 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/koel-koel#1880
No description provided.