Charcoal-SE / SmokeDetector

Headless chatbot that detects spam and posts links to it in chatrooms for quick deletion.
https://metasmoke.erwaysoftware.com
Apache License 2.0
474 stars 182 forks source link

Replying `n` to a chat message doesn't record feedback anymore #1362

Closed Glorfindel83 closed 6 years ago

Glorfindel83 commented 6 years ago

Case in point: https://chat.stackexchange.com/transcript/message/41641607#41641607

As quartata suggested, replying naa- does work, e.g. here.

quartata commented 6 years ago

This is a Metasmoke inconsistency -- normally with most feedbacks, only the first letter matters -- e.g. if it starts with t then it's a tp, if it starts with f then it's an fp But naa appears to have to be exactly naa.

ArtOfCode- commented 6 years ago

Metasmoke inconsistency, but still a Smokey bug :)

Glorfindel83 commented 6 years ago

I have a history of posting metasmoke bugs on the SmokeDetector project and vice versa ...

Anyway, the problem seems to extend to k as well: https://chat.stackexchange.com/transcript/message/41653240#41653240

angussidney commented 6 years ago

Ideally, for the sake of consistency we’d like to continue using the old behaviour. i.e. everything is extended to tpu-, naa- etc

quartata commented 6 years ago

The k not working was due to your notes after it which were interpreted as part of the command, that's a separate thing. I'll fix both later today.