mirror of
https://github.com/koel/koel.git
synced 2026-04-25 00:36:03 +03:00
[GH-ISSUE #380] Better management for cover images #274
Labels
No labels
Authentication
Dependencies
Documentation
Feature Request
Flac
Help Wanted
Installation/Setup
Integration
Mobile
PR Welcome
Pending Release
Performance
Playlist
S3
Search
Sync
[Pri] Low
[Pri] Normal
[Status] Keep Open
[Status] Needs Author Reply
[Status] Needs Review
[Status] Stale
[Status] Will Implement
[Type] Blessed
[Type] Bug
[Type] Duplicate
[Type] Enhancement
[Type] Help Request
[Type] Question
[Type] Task
pull-request
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/koel-koel#274
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
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?
Originally created by @ronilaukkarinen on GitHub (Jul 18, 2016).
Original GitHub issue: https://github.com/koel/koel/issues/380
Sorry if this is already asked, but didn't find with search. I have a large library and every folder has at least cover.jpg, folder.jpg or Cover.jpg. Some underground albums don't have covers at all from last.fm, so there's not any way to change them now.
Would it be possible to hover+point and click and change the cover from Koel UI or add/remove files from somewhere? Would be also cool for every scan for Koel to check if there's already a cover or folder jpg in the album directory or not, only if not, then try to fetch from Last.fm.
@phanan commented on GitHub (Jul 19, 2016):
Is there any (undocumented) rule regarding the file name? Or is
(cov|fold)er\.(jpe?g|png)/ialready enough?@ronilaukkarinen commented on GitHub (Jul 19, 2016):
Windows used folder.jpg so that's why people are usually including it with their albums, but some other apps use cover.jpg and I think Bandcamp as well. That's unwritten "rule", yes, if that's what you mean. At this point I think that regexp is enough.
@phanan commented on GitHub (Aug 7, 2016):
Implemented. If you're on
masterbranch, remember to resync withartisan koel:sync --force --tags=cover(and make a backup!)