search

SOBotics / FireAlarm

A Stack Exchange chatbot to catch low-quality posts.
https://bonfire.sobotics.org
MIT License
14 stars 8 forks source link

FireAlarm reports closed questions. #10

Closed double-fault closed 7 years ago

double-fault commented 7 years ago

The title explains it. Posting it here as a reminder for myself.

NobodyNada commented 7 years ago

IIRC it only does when the question is closed seconds before being reported, because network delay and API caching.

double-fault commented 7 years ago

@NobodyNada nope. Yesterday it posted a question which was closed one hour before the report. The post it reported was detected after an edit.

double-fault commented 7 years ago

uh, there seems to be a problem @NobodyNada . I was going to fix this today when I realised in the function checkAndReportPost a type Post is passed, not Question. It surely does not make sense making one more api call, so should I change it such that everywhere Question is passed instead of Post?

NobodyNada commented 7 years ago

@Asish sure!

double-fault commented 7 years ago

@NobodyNada ok, I'll do that tomorrow or day after.