[PR #1603] [MERGED] feat: allow specifying folder when create/edit playlists #1804

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

📋 Pull Request Information

Original PR: https://github.com/koel/koel/pull/1603
Author: @phanan
Created: 11/27/2022
Status: Merged
Merged: 11/27/2022
Merged by: @phanan

Base: masterHead: feat/playlist-specify-folder


📝 Commits (1)

  • 1017375 feat: allow specifying folder when create/edit playlists

📊 Changes

53 files changed (+1158 additions, -881 deletions)

View changed files

📝 app/Casts/SmartPlaylistRulesCast.php (+12 -8)
app/Exceptions/PlaylistBothSongsAndRulesProvidedException.php (+13 -0)
📝 app/Http/Controllers/API/Interaction/BatchLikeController.php (+2 -1)
📝 app/Http/Controllers/API/PlaylistController.php (+17 -8)
📝 app/Http/Controllers/API/PlaylistSongController.php (+4 -2)
📝 app/Http/Controllers/V6/API/GenreController.php (+1 -4)
📝 app/Http/Controllers/V6/API/GenreSongController.php (+1 -5)
📝 app/Http/Controllers/V6/API/PlaylistController.php (+40 -10)
📝 app/Http/Controllers/V6/API/QueueController.php (+9 -9)
📝 app/Http/Middleware/Authenticate.php (+1 -3)
📝 app/Http/Requests/API/PlaylistStoreRequest.php (+5 -2)
📝 app/Http/Requests/API/PlaylistUpdateRequest.php (+4 -0)
📝 app/Models/Playlist.php (+4 -4)
📝 app/Repositories/InteractionRepository.php (+6 -8)
📝 app/Rules/ValidSmartPlaylistRulePayload.php (+3 -10)
📝 app/Services/PlaylistService.php (+44 -13)
📝 app/Values/SmartPlaylistRuleGroup.php (+6 -12)
app/Values/SmartPlaylistRuleGroupCollection.php (+13 -0)
📝 resources/assets/js/components/layout/ModalWrapper.spec.ts (+2 -2)
📝 resources/assets/js/components/layout/ModalWrapper.vue (+23 -22)

...and 33 more files

📄 Description

No description provided


🔄 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/1603 **Author:** [@phanan](https://github.com/phanan) **Created:** 11/27/2022 **Status:** ✅ Merged **Merged:** 11/27/2022 **Merged by:** [@phanan](https://github.com/phanan) **Base:** `master` ← **Head:** `feat/playlist-specify-folder` --- ### 📝 Commits (1) - [`1017375`](https://github.com/koel/koel/commit/1017375d4584e5400db1f3fbd0b54384493dfed6) feat: allow specifying folder when create/edit playlists ### 📊 Changes **53 files changed** (+1158 additions, -881 deletions) <details> <summary>View changed files</summary> 📝 `app/Casts/SmartPlaylistRulesCast.php` (+12 -8) ➕ `app/Exceptions/PlaylistBothSongsAndRulesProvidedException.php` (+13 -0) 📝 `app/Http/Controllers/API/Interaction/BatchLikeController.php` (+2 -1) 📝 `app/Http/Controllers/API/PlaylistController.php` (+17 -8) 📝 `app/Http/Controllers/API/PlaylistSongController.php` (+4 -2) 📝 `app/Http/Controllers/V6/API/GenreController.php` (+1 -4) 📝 `app/Http/Controllers/V6/API/GenreSongController.php` (+1 -5) 📝 `app/Http/Controllers/V6/API/PlaylistController.php` (+40 -10) 📝 `app/Http/Controllers/V6/API/QueueController.php` (+9 -9) 📝 `app/Http/Middleware/Authenticate.php` (+1 -3) 📝 `app/Http/Requests/API/PlaylistStoreRequest.php` (+5 -2) 📝 `app/Http/Requests/API/PlaylistUpdateRequest.php` (+4 -0) 📝 `app/Models/Playlist.php` (+4 -4) 📝 `app/Repositories/InteractionRepository.php` (+6 -8) 📝 `app/Rules/ValidSmartPlaylistRulePayload.php` (+3 -10) 📝 `app/Services/PlaylistService.php` (+44 -13) 📝 `app/Values/SmartPlaylistRuleGroup.php` (+6 -12) ➕ `app/Values/SmartPlaylistRuleGroupCollection.php` (+13 -0) 📝 `resources/assets/js/components/layout/ModalWrapper.spec.ts` (+2 -2) 📝 `resources/assets/js/components/layout/ModalWrapper.vue` (+23 -22) _...and 33 more files_ </details> ### 📄 Description _No description provided_ --- <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:22 +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#1804
No description provided.