Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

lift limitation of 10000 tracks in a playlist #1600

Closed
lilmike opened this issue Feb 11, 2017 · 5 comments
Closed

lift limitation of 10000 tracks in a playlist #1600

lilmike opened this issue Feb 11, 2017 · 5 comments
Labels
C-enhancement Category: A PR with an enhancement or an issue with an enhancement proposal good first issue Call for participation
Milestone

Comments

@lilmike
Copy link

lilmike commented Feb 11, 2017

I had asked on the IRC channel a while back if there could be support for more than 10000 songs in a playlist, and was told it was to provide compatibility with mpd clients that can't handle more than that. Apparently there is some kind of MPD extension (though I can't seem to find where the details of it reside right now) that allows for that, and it's been implemented for a while. I was just opening this issue to get it in an official feature request form and to get more discussion on it.
-Michael.

@kingosticks
Copy link
Member

There's a core option called max_tracklist_length that controls this limit. What's wrong with you just setting this higher? https://docs.mopidy.com/en/latest/config/#confval-core/max_tracklist_length

@adamcik
Copy link
Member

adamcik commented Feb 11, 2017

The settings validator has max=10000 set. So someone just needs to remove that and update the docs.

@lilmike
Copy link
Author

lilmike commented Feb 11, 2017 via email

@adamcik
Copy link
Member

adamcik commented Feb 11, 2017

@jodal jodal added A-mpd C-enhancement Category: A PR with an enhancement or an issue with an enhancement proposal good first issue Call for participation labels Mar 31, 2018
@jodal jodal added this to the v2.2 milestone Apr 6, 2018
@jodal
Copy link
Member

jodal commented Apr 9, 2018

Fixed by #1666.

@jodal jodal closed this as completed Apr 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-enhancement Category: A PR with an enhancement or an issue with an enhancement proposal good first issue Call for participation
Projects
None yet
Development

No branches or pull requests

4 participants