Commands should not be case sensitive

Question

callumacrae opened this issue 12 years ago · comments

Xavier L. · Answer 1 · Fri Feb 17 2012 00:40:32 GMT+0800 (China Standard Time)

I'll work on it

Xavier L. · Answer 2 · Fri Feb 17 2012 01:05:57 GMT+0800 (China Standard Time)

Nevermind, I don't understand the code yet

Callum Macrae · Answer 3 · Fri Feb 17 2012 01:20:47 GMT+0800 (China Standard Time)

Line 460 of handler.js, the others are already case insensitive. Just replace cmd with cmd.toLowerCase().

Xavier L. · Answer 4 · Sun Feb 19 2012 02:22:10 GMT+0800 (China Standard Time)

see pull request number 33

Callum Macrae · Answer 5 · Mon Feb 20 2012 06:09:10 GMT+0800 (China Standard Time)

All we need now is for the various admin commands to lowercase commands as it adds them.

Callum Macrae · Answer 6 · Thu Mar 08 2012 21:37:10 GMT+0800 (China Standard Time)

@Xavier-L: Feel like making it lowercase commands when they're added?

Xavier L. · Answer 7 · Sat Mar 10 2012 00:37:19 GMT+0800 (China Standard Time)

Sure thing
On Mar 8, 2012 8:37 AM, "Callum Macrae" <
reply@reply.github.com>
wrote:

@Xavier-L: Feel like making it lowercase commands when they're added?

Reply to this email directly or view it on GitHub:
#20 (comment)