search

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

FR: Make SmokeDetector's "tpu- by <name>" message reply to the report. #573

Closed Cerbrus closed 7 years ago

Cerbrus commented 7 years ago

Just a small eye-candy modification:

I'd like to see SmokeDetector's feedback responses reply to the report the feedback is applied to. This will allow chat's message highlighting / navigation to link between the report and it's feedback. Especially practical for the feedback messages for older reports, or cases like this:

capture

ArtOfCode- commented 7 years ago

We don't have the data to do this with, and it doesn't seem like it's really worth getting it for something this small. The tpu- by <name> messages are sent from metasmoke to Smokey via a websocket. metasmoke doesn't know the message ID that the report is in, so it can't add it to the message; Smokey can't either because those messages aren't the only thing that comes down the socket, and other messages don't need to be replies.

Cerbrus commented 7 years ago

I was hoping a message Id was stored somewhere, but since that's not the case, I agree with "doesn't seem like it's really worth getting it". Too much effort for too little gain.