Error linking to SmokeDetector notice in chat for feedback

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

Error linking to SmokeDetector notice in chat for feedback #673

Closed tripleee closed 4 years ago

tripleee commented 7 years ago

It seems that linking back to a Smokey notice in chat for feedback gets resolved to the latest feedback to the post in question, so you can end up providing incorrect feedback for a post which has been edited.

For example, I posted feedback to vandalism which had been reverted. The rollbacks were also posted to chat because the user was blacklisted. Now, even though I specifically linked to the notices for the vandalized posts, the feedback was posted to the rollbacks.

Transcript: http://chat.stackoverflow.com/transcript/message/36808532#36808532

I have not removed my feedback from the posts so that the evidence is still there if you should need it.

teward commented 7 years ago

So, if a post is edited, you're saying it gets a duplicate record, and somehow the feedback can be given back to the wrong MS record? (It's unclear what exactly you're referencing...)

ArtOfCode- commented 7 years ago

@teward Imagine:

Message 1: SmokeDetector report of post A. Post A is a tpu.
Message 2: tpu- by User A
Message 3: SmokeDetector report of post A, for the second time because it got caught again. Requires different feedback this time, say fp-.
Message 4 (reply to 1), by User B: tpu-

User B's tpu- is now attached to the second report of Post A in metasmoke.

tripleee commented 7 years ago

Yes, exactly. Should I update the problem description or is this sufficient?

stale[bot] commented 4 years ago

This issue has been closed because it has had no recent activity. If this is still important, please add another comment and find someone with write permissions to reopen the issue. Thank you for your contributions.