-
-
Notifications
You must be signed in to change notification settings - Fork 2.7k
Voice channel mention #932
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
Conversation
I recently noticed voice channels can be mentioned just like text channels. A small change to the code to implement this feature.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
JDA offers a method to get a channel's mention string. Use that method instead.
src/main/java/com/jagrosh/jmusicbot/commands/admin/SetvcCmd.java
Outdated
Show resolved
Hide resolved
Additionally, my IDE has found another 3 usages of the voice channel name that you've missed:
Please update these as well. |
Thanks for the elaborate feedback. After successfully compiling the project, I ran the commands and got some good results, a few of which you can see below: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! @jagrosh will have to take another look & merge if everything looks good.
* Voice channel mention I recently noticed voice channels can be mentioned just like text channels. A small change to the code to implement this feature. * Voice channel mention * vc.getName() changed to vc.getAsMention()
This pull request...
Description
I recently noticed voice channels can be mentioned just like text channels. A small change to the code can implement this feature.
The top example is the current way the bot would mention a channel, the bottom is what I wish it would look like.
This is my first pull request. I have not tested this change as I consider it to be too simple and don't expect it to fail.
Purpose
Aesthetics and ease of use: clicking on the mentioned channel will automatically connect you to it.