Feature Request: Limit Song Requests to Playlist

[parendev]

Feature Request: Limit Song Requests to Playlist

[Inzaniity]

Technically this can be done. The issue is, that every time a song request is made I'd have to fetch the entire playlist again and again to compare. This adds slight processing time and can put a strain on the rate limit. I could cache the playlist after the first iteration but then if you'd add a song to the playlist mid-stream I also need a way to manually update.

Limiting the lookup is not possible I think since it's a direct API endpoint on Spotify's end. The only thing I can do is compare the track id of the song request to the playlist and if it's not in there respond with "nope".

In theory, this sounds manageable to implement, but I can't promise. I'm currently taking a breather from the app, but I will think about it!

[parendev]

Can definitely appreciate taking a break. :)

I looked into the Spotify API briefly before opening this ticket and saw the same search endpoint problem. It seemed to me that to get the playlist-constrained search Songify would need to implement the search functionality itself against a cached copy of the playlist's songs.

Since it sounds like that would need to be cached anyways, I'd think a very rudimentary search (e.g. case-normalized exact match) against the cache before falling back to the search API probably wouldn't be too bad. The playlist search functionality wouldn't exactly match Spotify's search, but I'm not sure that matters.

That said, I'm not sure how big of a problem false matches against copies of the songs outside the playlist even is. Nor how effective an exact match lookup from the cache would be. Feels a bit like a slippery slope where if exact match isn't good enough it rapidly becomes more difficult. Search is hard. Might not be worth adding in any form.

Just figured I'd share my brainstorming for if/when you revisit this. :)

[Inzaniity]

I wrote a first concept of this and it seems to work. It needs more testing but I think it'll be possible :)

Keep in mind that the playlist has to be one that you created.

[Inzaniity]

Implemented and released in v1.5.0

[parendev]

Dude, I had missed the notifications for this until now. You're amazing. Thank you so much!