Add paging through fetched playlists

amcquade / fresh_script

Find Spotify tracks posted to the HipHopHeads subreddit and add them to a Spotify playlist.

MIT License

91 stars 61 forks source link

Add paging through fetched playlists #47

Closed adamfitzgibbon closed 5 years ago

adamfitzgibbon commented 5 years ago

addresses #44
uses the offset parameter to page through additional playlists, should the user have more than 50 followed playlists
this ended up being a bit more complex than I would have liked it, so it ended up being a little messy. Feel free to comment with ways to clean these changes up!

adamfitzgibbon commented 5 years ago

Hey sorry I got a bit busy but I'll still address this possibly this weekend. I'm only displaying playlists that you own. Are you testing with your own playlists or playlists that you follow?

amcquade commented 5 years ago

@adamfitzgibbon not a problem. displaying playlists that you follow wouldnt make that much sense because you wouldnt be able to add songs to it. i can try again but i couldnt get the script to list any additional playlists that i own.

adamfitzgibbon commented 5 years ago

I added the ability to quit out of the loop. I tested and was still able to cycle through playlists. Can you describe the error a little more? Admittedly I don't have that many playlists, but when I test I reduce the limit on how many are fetched so that I can see the paging logic working.

adamfitzgibbon commented 5 years ago

Also, I didn't realize that you added a GUI. Sweet! I didn't have time to get flask setup tonight but I'll do that as well so I can test with that as well.

amcquade commented 5 years ago

@adamfitzgibbon sounds good, ill try to test that code this weekend. let me know if you have any issues setting up flask

adamfitzgibbon commented 5 years ago

I got flask setup it was pretty easy. I'll probably leave setting up the playlist GUI for a separate ticket because I think it would add a lot of complexity to this PR.