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

Permissions #84

Open
LunarSera opened this issue Dec 23, 2018 · 7 comments
Open

Permissions #84

LunarSera opened this issue Dec 23, 2018 · 7 comments

Comments

@LunarSera
Copy link

Please escape mass mentions in the Permissions cog, regarding the roles function

@skeith
Copy link
Contributor

skeith commented Dec 23, 2018

you don't need to escape massmention if you do it on a channel where no one can see but you

you can also use the nomassmention cog for that

@LunarSera
Copy link
Author

Neither of those solutions are even real solutions. And I shouldn't have to patch a bot because of bad code.
It's not hard to edit the code to escape mass mentions.
And I'm not the owner of the server, so there are other admins that can see every channel. Not that I'd support your idea anyway.
Any code that allows the bot to. Mention everyone without intention is bad code. Plain and simple.
By bad code I mean, needs fixed. Not that it's actually bad.

@skeith
Copy link
Contributor

skeith commented Dec 23, 2018

Did you miss

you can also use the nomassmention cog for that

@LunarSera
Copy link
Author

I saw it. I chose to. Ignore it and say that I shouldn't have to patch the bot because of bad code. That cog shouldn't even need to exist.

@skeith
Copy link
Contributor

skeith commented Dec 23, 2018

Well, I tried to help

it's your choice

@LunarSera
Copy link
Author

I reported a bug. That's the idea. It needs fixed. I wasn't looking for a patch

@skeith
Copy link
Contributor

skeith commented Dec 23, 2018

That wouldn't be so soon, since Will paused this repo progress to concentrate on Red v3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants