Page MenuHomePhabricator

mw-ext-ORES should play nice with patrol flags
Closed, ResolvedPublic


It should be possible to clear the "r" flag somehow, after manual intervention. One alternative might be to respect the rc_patrolled flag in our filter, so "r" disappears once the edit has been reviewed. In any case, we should not require that editors perform two actions to clear both the patrolled and "r" flags.

Event Timeline

awight created this task.Dec 15 2015, 11:07 PM
awight raised the priority of this task from to Needs Triage.
awight updated the task description. (Show Details)
awight added a subscriber: awight.
Restricted Application added subscribers: StudiesWorld, Aklapper. · View Herald TranscriptDec 15 2015, 11:07 PM

I talked with @Halfak about this and I thought playing around with patrol status would be best option.
Creating several patrol status would be annoying for users but when a patrol marks an edit as patrolled, we don't show the "r" sign anymore. It can be pretty helpful.

There should be a two conditions for showing the "r" sign. 1- Score passing the threshold 2- it's not patrolled. Then we can query the database and find out our false positives much easier.

It also can be something optional, let user choose when "r" will be showed, maybe someone doesn't trust the patrol system. (we can let user determine the threshold too but it's not in scope of this task obviously)

Change 264293 had a related patch set uploaded (by Ladsgroup):
Don't show the r flag when edit is patrolled

awight closed this task as Resolved.Jan 17 2016, 4:48 PM
awight claimed this task.

Verified working!