Skip to content
This repository has been archived by the owner on Dec 24, 2021. It is now read-only.

Add unhandledErrorResponse option, to ignore unhandled errors #146

Closed
wants to merge 2 commits into from

Conversation

danbopes
Copy link

This is in regards to #145. Adds additional parameter to CommandoClient options: unhandledErrorResponse

To ensure backwards compatibility, this defaults to true. If unhandledErrorResponse is set to false, a message will return null (no response) if an unhandled error occurs, and allows the commandError emit event to handle the unhandled error.

…mandoClient options: unhandledErrorResponse

Ensure backwards compatibility, defaults to true. If unhandledErorrRepsponse is set to false, a message will return null (no response) if an unhandled error occurs, and allows the commandError emit event to handle the unhandled error.
@lilyissillyyy
Copy link
Contributor

Maybe commandErrorResponse instead, to keep with the event name.

@Gawdl3y
Copy link
Member

Gawdl3y commented May 21, 2018

I don't think this is the right way to go about doing this - the translations branch will take care of the customisation of this message.

@danbopes
Copy link
Author

Is it going to be easy to change one message from the current locale set? This seems like a much more difficult way to go, for a user who just wants a little customization.

@lilyissillyyy
Copy link
Contributor

It should be pretty simple, all you would have to do is extend Commando's locale set and change one entry.

@Gawdl3y
Copy link
Member

Gawdl3y commented Sep 26, 2018

This would be better-addressed by #193 and #197.

@Gawdl3y Gawdl3y closed this Sep 26, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants