Skip to content
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

Mentioning @everyone does not work in the other channels. #18

Open
JSai23 opened this issue Nov 2, 2020 · 21 comments
Open

Mentioning @everyone does not work in the other channels. #18

JSai23 opened this issue Nov 2, 2020 · 21 comments
Labels
bug Something isn't working

Comments

@JSai23
Copy link

JSai23 commented Nov 2, 2020

No description provided.

@bsian03
Copy link
Contributor

bsian03 commented Nov 2, 2020

NAB? I don't see how that would be helpful

@bsian03
Copy link
Contributor

bsian03 commented Nov 2, 2020

Happens due to allowed mentions

@JSai23
Copy link
Author

JSai23 commented Nov 2, 2020

no i gave the bot full admin

@JSai23
Copy link
Author

JSai23 commented Nov 2, 2020

its not doing any mentions

@JSai23
Copy link
Author

JSai23 commented Nov 2, 2020

image

@JSai23
Copy link
Author

JSai23 commented Nov 2, 2020

image

@JSai23
Copy link
Author

JSai23 commented Nov 2, 2020

image

@JSai23
Copy link
Author

JSai23 commented Nov 2, 2020

so in the second server it doesnt mention everyone even tho it has full permissions and is an admin
I've tried with and without enhanced mention. Changed to code to not parse at all and nothing has fixed it
everything else has worked well for me.

@Khaaz
Copy link
Owner

Khaaz commented Nov 2, 2020

As Bsian stated it's because of allowedMentions, a change made by discord to mention that will block everyone mention unless specified.
The bot blocks everyone mention to prevent abuse by default.
If you really need to be able to mention everyone, I can tell which line to change.

@Khaaz
Copy link
Owner

Khaaz commented Nov 2, 2020

Also seeing your config, #17 will be solved by using the role ids and not the role name as stated in the issue.

@JSai23
Copy link
Author

JSai23 commented Nov 2, 2020

yeah figured out number 17
please if u could tell me the line to change

@JSai23
Copy link
Author

JSai23 commented Nov 2, 2020

ill be so grateful im trying to use this for moderating multiple servers from a centralized server and have modified it to fit my purposes this is the one thing I'm stuck with now.

@JSai23
Copy link
Author

JSai23 commented Nov 2, 2020

one more update even with enhanced mention turned on no mentions are working

@JSai23
Copy link
Author

JSai23 commented Nov 2, 2020

the bot is an admin in all servers and has all permissions

@Khaaz
Copy link
Owner

Khaaz commented Nov 2, 2020

I'll need to do some testing for mentions and I'll probably patch that directly in this repo. I'll keep you in touch directly in this issue when it's solved.

@JSai23
Copy link
Author

JSai23 commented Nov 8, 2020

Any updates on this. Thanks. Even if you just tell me which part is stopping the mentions I could fix it. I looked through the code couldn't figure out where.

@Khaaz
Copy link
Owner

Khaaz commented Nov 8, 2020

haven't had the time to get around that.
It's due to the allowedMentions property to pass around when executing a webhook (see here)

@JSai23
Copy link
Author

JSai23 commented Nov 10, 2020

can you tell me which line of the code

@JSai23
Copy link
Author

JSai23 commented Nov 10, 2020

update: i got the bot to tag everyone but it now sends the message twice once with the tag once as plain text
something to do with the enhance mention function
working on that now

@JSai23
Copy link
Author

JSai23 commented Nov 10, 2020

update: for my purposes, i didn't need enhanced mention so I got rid of it since it was causing the double @everyone. For now its working for my purposes fully. I use it as a bot for common announcements and trading signals across a multitude of servers from a master server so.

@Khaaz
Copy link
Owner

Khaaz commented Nov 11, 2020

thanks for the update.
I'll still need to update the bot to make mentions work correctly out of the box so I'll let the issue open for now.
Glad you managed to make it work like you expected!

@Khaaz Khaaz added the bug Something isn't working label Dec 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants