Page MenuHomePhabricator

Improve mwext-ORES integration with RecentChanges
Closed, ResolvedPublic

Description

This is a follow-on iteration of T112856's Recent Changes hooks, starting with these known bugs:

  • The "R" flag should hide once a human has reviewed the offending change.
  • Fix mw-core "old" recent changes view to respect recentChangesFlags.
  • Add a link to suspected damaging change lines, even when the review flag is hidden. The link text summarizes the perceived issue and the link target is this revision's revscore details page.
  • TODOs in code.

Event Timeline

awight created this task.Dec 3 2015, 8:13 AM
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 3 2015, 8:13 AM
awight renamed this task from Improve RecentChanges to Improve ORES integration with RecentChanges.Dec 3 2015, 8:21 AM
awight updated the task description. (Show Details)
awight set Security to None.
awight renamed this task from Improve ORES integration with RecentChanges to Improve mwext-ORES integration with RecentChanges.Dec 9 2015, 10:05 AM
awight updated the task description. (Show Details)

This is a follow-on iteration of T112856's Recent Changes hooks, starting with these known bugs:

  • The "R" flag should hide once a human has reviewed the offending change.

It's being done via using the patrol flag

  • Fix mw-core "old" recent changes view to respect recentChangesFlags.

Done a very very long time ago

  • Add a link to suspected damaging change lines, even when the review flag is hidden. The link text summarizes the perceived issue and the link target is this revision's revscore details page.

Nice idea, we should have a dedicated phab card

  • TODOs in code.

Done and removed the TODO