Zirak / SO-ChatBot

Other
124 stars 60 forks source link

Have the tell command default to using the echo command? #178

Closed AmaanC closed 10 years ago

AmaanC commented 10 years ago

I see a lot of people using tell like this:

!!tell userName I want this to be said from the bot

Recent example:

!!tell ossek you're welcome

Maybe if no command is specified to the !!tell command, we should default to it using echo since that seems to be what users assume?

ralt commented 10 years ago

:+1:

benjamingr commented 10 years ago

:+1:

Zirak commented 10 years ago

Note that /echo is not a builtin command. This is by design, and here's why: It gives you a touch of the bot. /echo is important functionality, which can oh-so-easily be done with /learn. So, instead of giving it to everyone who boots up the bot, why not make them work a bit for it? /tell is also a very important builtin. The command parameter is often confusing (I know), and people usually forget it, which gives me two reasons to not like this proposal:

  1. If you want the bot to link Joe to the mdn page on border-radius, and you accidentally do !!tell Joe border-radius, what you'll get is an error. Under your proposal, we'll get @Joe border-radius
  2. It forces people to learn the syntax, thereby avoiding (or at least limiting) problem 1.

I can agree to changing /tells error message, telling people the complete syntax and that it's not an echo command.

SomeKittens commented 10 years ago

I like changing the error message.

AmaanC commented 10 years ago

Good points. Let's just change the error, then On Feb 21, 2014 4:58 AM, "Randall Koutnik" notifications@github.com wrote:

I like changing the error message.

Reply to this email directly or view it on GitHubhttps://github.com/Zirak/SO-ChatBot/issues/178#issuecomment-35682382 .