search

nukeop / nuclear

Streaming music player that finds free music for you
https://nuclear.js.org/
GNU Affero General Public License v3.0
11.72k stars 1.02k forks source link

Add all tracks in the queue to an existing playlist #1577

Closed rohanrider closed 2 months ago

rohanrider commented 3 months ago

It would be awesome to add all tracks in the queue to an existing playlist (like you can already do to create a new playlist). Love the software. Thanks for it!

nukeop commented 3 months ago

Thanks! That's a pretty natural option to have.

For now, advanced users can merge playlists manually in the config json, but there should be a button added for this.

sossost commented 2 months ago

Hi ! @nukeop I am sossost, a new contributor. I'd like to contribute to this issue! Can I proceed??

sferra commented 2 months ago

Whoever ends up taking up this issue, please check AlbumView which contains functionality for adding an album to an existing or a new playlist. Adding the queue items to a playlist should be quite similar.

nukeop commented 2 months ago

Hi ! @nukeop I am sossost, a new contributor. I'd like to contribute to this issue! Can I proceed??

Sure, would you like to take both of those issues?

sossost commented 2 months ago

@nukeop oh! No, I will choose this issue. thank you!

sossost commented 2 months ago

Thank you! I'm glad to hear that. Your feedback motivates me to maintain high standards in my work. I'll keep up the good work!"

nukeop commented 2 months ago

Thank you, that was really quite awesome. You did everything - set up the project for yourself, implemented the feature very well, created a test for it, and updated existing tests, committing only the necessary changes. Very nice!