[PR #1947] [MERGED] feat: use public IDs for albums and artists #1971

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

📋 Pull Request Information

Original PR: https://github.com/koel/koel/pull/1947
Author: @phanan
Created: 6/13/2025
Status: Merged
Merged: 6/14/2025
Merged by: @phanan

Base: masterHead: artist-album-pubid


📝 Commits (1)

  • b14fefb feat: use public ID for albums and artists

📊 Changes

81 files changed (+320 additions, -202 deletions)

View changed files

📝 app/Enums/PermissionableResourceType.php (+1 -1)
📝 app/Helpers/Uuid.php (+2 -0)
📝 app/Http/Controllers/API/PlaylistCollaboration/PlaylistCollaboratorController.php (+1 -1)
📝 app/Http/Requests/API/PlaylistCollaboration/PlaylistCollaboratorDestroyRequest.php (+2 -2)
📝 app/Http/Resources/AlbumResource.php (+2 -2)
📝 app/Http/Resources/ArtistResource.php (+1 -1)
📝 app/Http/Resources/CollaborativeSongResource.php (+1 -1)
📝 app/Http/Resources/PlaylistCollaboratorResource.php (+1 -1)
📝 app/Http/Resources/SongResource.php (+3 -3)
📝 app/Models/Album.php (+7 -1)
📝 app/Models/Artist.php (+7 -1)
app/Models/Contracts/PermissionableResource.php (+8 -0)
📝 app/Models/Song.php (+1 -3)
📝 app/Providers/AppServiceProvider.php (+4 -0)
📝 app/Repositories/SongRepository.php (+2 -2)
📝 app/Repositories/UserRepository.php (+5 -0)
📝 app/Services/ResourcePermissionService.php (+11 -3)
📝 app/Values/PlaylistCollaborator.php (+5 -5)
📝 package.json (+1 -1)
📝 resources/assets/js/__tests__/factory/albumFactory.ts (+4 -5)

...and 61 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/1947 **Author:** [@phanan](https://github.com/phanan) **Created:** 6/13/2025 **Status:** ✅ Merged **Merged:** 6/14/2025 **Merged by:** [@phanan](https://github.com/phanan) **Base:** `master` ← **Head:** `artist-album-pubid` --- ### 📝 Commits (1) - [`b14fefb`](https://github.com/koel/koel/commit/b14fefb36e18992eb8fe7f8ddc464536573fa93b) feat: use public ID for albums and artists ### 📊 Changes **81 files changed** (+320 additions, -202 deletions) <details> <summary>View changed files</summary> 📝 `app/Enums/PermissionableResourceType.php` (+1 -1) 📝 `app/Helpers/Uuid.php` (+2 -0) 📝 `app/Http/Controllers/API/PlaylistCollaboration/PlaylistCollaboratorController.php` (+1 -1) 📝 `app/Http/Requests/API/PlaylistCollaboration/PlaylistCollaboratorDestroyRequest.php` (+2 -2) 📝 `app/Http/Resources/AlbumResource.php` (+2 -2) 📝 `app/Http/Resources/ArtistResource.php` (+1 -1) 📝 `app/Http/Resources/CollaborativeSongResource.php` (+1 -1) 📝 `app/Http/Resources/PlaylistCollaboratorResource.php` (+1 -1) 📝 `app/Http/Resources/SongResource.php` (+3 -3) 📝 `app/Models/Album.php` (+7 -1) 📝 `app/Models/Artist.php` (+7 -1) ➕ `app/Models/Contracts/PermissionableResource.php` (+8 -0) 📝 `app/Models/Song.php` (+1 -3) 📝 `app/Providers/AppServiceProvider.php` (+4 -0) 📝 `app/Repositories/SongRepository.php` (+2 -2) 📝 `app/Repositories/UserRepository.php` (+5 -0) 📝 `app/Services/ResourcePermissionService.php` (+11 -3) 📝 `app/Values/PlaylistCollaborator.php` (+5 -5) 📝 `package.json` (+1 -1) 📝 `resources/assets/js/__tests__/factory/albumFactory.ts` (+4 -5) _...and 61 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:55 +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#1971
No description provided.