search

jakevoytko / crbot

A Discord bot that acts as a call-and-response bot. It reacts to ? commands with the learned response.
MIT License
7 stars 3 forks source link

In ?list command show if argument is required. #14

Closed likwidoxigen closed 7 years ago

likwidoxigen commented 7 years ago

The list command does not specify which commands require arguments. It makes for having to test unknowns in private message or risk looking foolish.

drvan commented 7 years ago

Would changing ?list to return something like the following help?

List of builtins:
 - ?help
 - ?learn <call> <response>
 - ?list
 - ?unlearn <call>
likwidoxigen commented 7 years ago

As long as it tells me something like ?nou So I know to use ?nou @arc Instead of getting an error.

On Apr 10, 2017 7:38 PM, "Ryan Van Antwerp" notifications@github.com wrote:

Would changing ?list to return something like the following help?

List of builtins:

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/jakevoytko/crbot/issues/14#issuecomment-293108768, or mute the thread https://github.com/notifications/unsubscribe-auth/AAV3VHJm9GFl9HkNhisVIAhZkDIvLZe1ks5rur1bgaJpZM4M3HiH .

drvan commented 7 years ago

Oh - you're not referring to the ?list commands themselves, but rather all learned commands - you'd like to know if '?nou' requires and argument or not. I'll take a look and see what the LoE to implement would be.

likwidoxigen commented 7 years ago

Awesome! Thank you!

On Apr 10, 2017 10:18 PM, "Ryan Van Antwerp" notifications@github.com wrote:

Oh - you're not referring to the ?list commands themselves, but rather all learned commands - you'd like to know if '?nou' requires and argument or not. I'll take a look and see what the LoE to implement would be.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/jakevoytko/crbot/issues/14#issuecomment-293130605, or mute the thread https://github.com/notifications/unsubscribe-auth/AAV3VI_wmAenK-pD7whbgjC-XS38PNoQks5ruuMKgaJpZM4M3HiH .

jakevoytko commented 7 years ago

@likwidoxigen do you also want ?help to list this information for custom commands, or would you only find it useful to have this in ?list?

jakevoytko commented 7 years ago

I'm bored, and as far as I can tell, Bryce has been raptured (or something). Accordingly, I'm making a unilateral decision to append help text to each of the commands in ?list. Additionally, ?help is changing so that it prints help text for custom commands

jakevoytko commented 7 years ago

Fixed by e6f49accf606109fe45f183cc449dc331ae40a30

likwidoxigen commented 7 years ago

This sounds like it solves my issue. Thanks!

On Apr 16, 2017 13:42, "Jake Voytko" notifications@github.com wrote:

Fixed by e6f49ac https://github.com/jakevoytko/crbot/commit/e6f49accf606109fe45f183cc449dc331ae40a30

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/jakevoytko/crbot/issues/14#issuecomment-294364307, or mute the thread https://github.com/notifications/unsubscribe-auth/AAV3VCzMjjLpeU_-FQmCsiRrQ-A3lsN1ks5rwlMVgaJpZM4M3HiH .