Red-DiscordBot icon indicating copy to clipboard operation
Red-DiscordBot copied to clipboard

Make CommandInvokeError error message customisable

Open PredaaA opened this issue 2 years ago • 3 comments

What component of Red (cog, command, API) would you like to see improvements on?

Core

Describe the enhancement you're suggesting.

Currently when a command errors out, it sends the following message to all users: image

However "Check your console or logs for details." for end users doesn't make much sense, just for the bot owner(s) though it makes sense. My suggestion would be to make this message customisable, like we do on disabled commands. And by default let that message. Maybe some templating to send the command name, and "Check your console or logs for details." if the command invoker was the owner too.

Anything else?

No response

PredaaA avatar Mar 16 '22 21:03 PredaaA

I'll be looking into implementing this feature with @jevon64. Please let me know if you would like anything else to be considered! 😃

matcha19 avatar Mar 24 '22 21:03 matcha19

@matcha19 I would recommend against working on this particular issue at this time. This is a feature request which has not yet been accepted as something we are interested in pursuing. In particular, there are multiple considerations we would need to make before a PR could be made (what should the command name be? in what cog should it be located, or should it be in core? what specific customization should be allowed by this command?)

If you are looking for issues to help out with, any labeled with PRs Welcome are your best bet, though Accepted ones are generally also OK to work on assuming someone else has not already started work. Anything with Needs Triage has not been looked over by us yet, and given that our focus is currently on preparing the bot for 3.5, most likely will not be looked at in the near future.

Flame442 avatar Mar 24 '22 21:03 Flame442

@Flame442 Ah sounds good! We'll look at PR Welcome issues in the meantime. Thanks for letting us know.

matcha19 avatar Mar 24 '22 21:03 matcha19

Is it possible to have a setting for it to not spit out any error message at all? It really serves no purpose for the end user.

lakai avatar Jan 07 '23 17:01 lakai

I think it makes no sense to not send any message at all, it's bad UX. User should always be getting some response and at the last resort (unexpected exception) it should be the error about an issue with the command.

Jackenmen avatar Jan 07 '23 22:01 Jackenmen