Multi-line learning is broken

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

Multi-line learning is broken #16

Closed aetimmes closed 7 years ago

aetimmes commented 7 years ago

Steps to reproduce:

[9:48 PM] Timmes: ?learn was What a save! What a save! What a save! Chat disabled for 1 second. What a save! Chat disabled for 4 seconds.

Expected output:

[9:48 PM] DaaSbot: Learned about was [9:48 PM] Timmes: ?was [9:48 PM] DaaSbot: What a save! What a save! What a save! Chat disabled for 1 second. What a save! Chat disabled for 4 seconds.

Actual output:

[9:48 PM] DaaSbot: Learned about was What [9:48 PM] Timmes: ?was [9:48 PM] Timmes: uhhhhh bug report +Jake +Arc

drvan commented 7 years ago

Is it ever. I'm not entirely sure what happened, but it looks like part of the response was interpreted as part of the call. Getting a list of commands returns the following:

 - ?war
 - ?was
What
 - ?whitney

Calling ?was does nothing. @jakevoytko can you take a look at Redis to see what's actually stored there?

jakevoytko commented 7 years ago

Fixed by b5128f315d765de3557de1d1b35c29bb1411b616

aetimmes commented 7 years ago

Thanks all!

On Tue, Apr 11, 2017, 10:02 PM Jake Voytko notifications@github.com wrote:

Closed #16 https://github.com/jakevoytko/crbot/issues/16.

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