search

PredaaA / predacogs

Cogs for Red Discordbot
MIT License
30 stars 25 forks source link

[martools] command conflict #48

Closed phenom4n4n closed 3 years ago

phenom4n4n commented 4 years ago

Renames martools commands to be prefixed with m if they are conflicting, such as mprefix.

Drapersniper commented 4 years ago

This is something I'm not entirely in favour as a QA, this dynamically edit commands names making for a really inconsistent and confusing user experience.

ltzmax commented 4 years ago

this will just be confusing alot of people who have a command or 2 of what martools have.. so i don't even see this as a good idea. but thats up to preda anyway.

PredaaA commented 3 years ago

I'll close this PR, for reasons said on the comments above. I'm actually not keen of changing commands name on loading, I see that very confusing for users. But still thanks for your PR.