Khaaz / Cross-Server-Bot

Cross-Server bot - Link discord channels between them across servers.
MIT License
34 stars 16 forks source link

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

Open JSai23 opened 3 years ago

bsian03 commented 3 years ago

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

bsian03 commented 3 years ago

Happens due to allowed mentions

JSai23 commented 3 years ago

no i gave the bot full admin

JSai23 commented 3 years ago

its not doing any mentions

JSai23 commented 3 years ago

image

JSai23 commented 3 years ago

image

JSai23 commented 3 years ago

image

JSai23 commented 3 years ago

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 commented 3 years ago

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 commented 3 years ago

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

JSai23 commented 3 years ago

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

JSai23 commented 3 years ago

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 commented 3 years ago

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

JSai23 commented 3 years ago

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

Khaaz commented 3 years ago

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 commented 3 years ago

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 commented 3 years ago

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 commented 3 years ago

can you tell me which line of the code

JSai23 commented 3 years ago

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 commented 3 years ago

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 commented 3 years ago

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!