busty icon indicating copy to clipboard operation
busty copied to clipboard

Bold the title of the currently playing song in the pinned song listing

Open anoadragon453 opened this issue 2 years ago • 10 comments

Busty currently changes her name to the currently playing song, so that it's easy to see at a glance.

As an additional nice-to-have, we could also bold the title of the currently playing song in the song listing message. Like so:

image

This would require editing the song list message when a song starts playing. We could factor out the code that generates the contents of the !list command into a new function. This function would take the index of the currently playing song, and bold that song's filename in the output.

anoadragon453 avatar Jan 15 '22 15:01 anoadragon453

Hi @anoadragon453 can i work on this?

DipayanP007 avatar Jan 15 '22 19:01 DipayanP007

Hi @anoadragon453 can i work on this?

Hey, if you'd like to work on this then go right ahead! How did you find this project by the way?

Cephian avatar Jan 15 '22 20:01 Cephian

I really found the project interesting and innovative. And for the record I am workingon #25 too which is partly solved. So assign me that and I will get it done 😄

DipayanP007 avatar Jan 15 '22 20:01 DipayanP007

I really found the project interesting and innovative. And for the record I am workingon #25 too which is partly solved. So assign me that and I will get it done smile

Sure, if you'd like to take that on as well I don't believe anyone else is working on it

Cephian avatar Jan 15 '22 23:01 Cephian

Thanks for the PR @DipayanP007! We'll get to reviewing it very soon.

In the meantime, we're really grateful for your help on this project. It's allowed us to move faster on it. Given that, would you be interested in joining the Discord server that this bot is used in? It's primarily a place where people create and share music that they make every 2 weeks in a friendly competition scenario. After submissions are due, we all jump in a voice call and listen to what people have created!

If you're interested, let us know and we'll share an invite. If not, no worries!

anoadragon453 avatar Jan 17 '22 17:01 anoadragon453

Hey, I would love to be part of the server. But for your information, I am not that active on discord. If it's still okay with you, you can share the invite.

DipayanP007 avatar Jan 17 '22 17:01 DipayanP007

@DipayanP007 no worries! Participation is up to you: discord . gg / tJTC Q7kB (remove spaces)

anoadragon453 avatar Jan 18 '22 00:01 anoadragon453

Hey @anoadragon453 !list now does not work when a song is being played. So do you guys still need this feature?

DipayanP007 avatar Jan 27 '22 04:01 DipayanP007

You might be a little confused about what the !list command it supposed to do. It's only supposed to run once to gather and list all the songs, then pin those messages. The idea is that if someone wants to see what song is currently playing and where it is in the list, they will look at the pinned messages and see it there. So the bot should keep track of the pinned list message(s) we sent at the start, and then edit them to bold/unbold a song each time one starts/stops playing.

The behavior might seem a little strange, but it's built that way because it works the best for the server we made it for, where people send songs they made to a discord channel and every two weeks we play them all in one sitting for everyone to hear. If this behavior is too different from what you wanted originally to implement that's fine, just let us know. We're not in a rush to get this feature implemented either way.

Cephian avatar Jan 27 '22 04:01 Cephian

I had the wrong idea about the !list command. But no worries I will work on this. The project is intriguing. :smile:

DipayanP007 avatar Jan 27 '22 05:01 DipayanP007