Page MenuHomePhabricator
Create Task
Maniphest T94451

A3. Implement auto-patrolling for Flow
Closed, ResolvedPublic1 Estimated Story Points
Actions

Description

When a user with autopatrolled makes a revision, it should be marked patrolled.

Details

ProjectBranchLines +/-Subject
mediawiki/extensions/Flowmaster+2 -1Mark posts by autopatrolled users as patrolled
Customize query in gerrit

Related Objects
Search...

Event Timeline

• Mattflaschen-WMF created this task.Mar 30 2015, 6:30 PM
• Mattflaschen-WMF raised the priority of this task from to Needs Triage.
• Mattflaschen-WMF updated the task description. (Show Details)
• Mattflaschen-WMF added projects: Collaboration-Team-Triage, StructuredDiscussions.
• Mattflaschen-WMF added subscribers: Qgil, Amire80, EBernhardson and 6 others.
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptMar 30 2015, 6:30 PM
• Mattflaschen-WMF updated the task description. (Show Details)Mar 30 2015, 6:32 PM
• Mattflaschen-WMF set Security to None.
DannyH edited a custom field.Mar 30 2015, 6:32 PM
He7d3r added a subscriber: He7d3r.Apr 1 2015, 10:45 PM
Qgil added a comment.Apr 6 2015, 10:32 AM

Just a note to say that from all the subtasks of T73456, this one is which is causing more annoyance nowadays. It is probably fair to say that regular Flow users are likely to be autopatrolled users.

He7d3r added a comment.Apr 6 2015, 3:23 PM

Aren't they marked as patrolled already? It looks so from the screenshot F108661 I uploaded to T95025: C4. Fix watchlist patrolling rendering for Flow in regular watchlist (works in grouped watchlist).

EBernhardson triaged this task as Medium priority.Apr 6 2015, 5:54 PM
Qgil added a comment.Apr 7 2015, 1:04 PM

I just checked ca.wiki's recent changes and my watchlist, and both show a non-patrolled post from user Sorenike (autopatrolled).

DannyH added a comment.Apr 9 2015, 12:30 AM

Thanks for the info. I'll get this estimated and into a sprint...

Restricted Application added a project: Collaboration-Team-Triage. · View Herald TranscriptApr 22 2015, 7:09 PM
DannyH renamed this task from Implement auto-patrolling for Flow to A3. Implement auto-patrolling for Flow.Apr 22 2015, 8:04 PM
Catrope claimed this task.Apr 29 2015, 10:51 PM
gerritbot added a subscriber: gerritbot.Apr 30 2015, 12:04 AM

Change 207685 had a related patch set uploaded (by Catrope):
Mark posts by autopatrolled users as patrolled

https://gerrit.wikimedia.org/r/207685

Qgil added a comment.Apr 30 2015, 6:13 AM

Catrope claimed this task.

The reorg is working! \o/

:)

gerritbot added a comment.Apr 30 2015, 5:36 PM

Change 207685 merged by jenkins-bot:
Mark posts by autopatrolled users as patrolled

https://gerrit.wikimedia.org/r/207685

DannyH closed this task as Resolved.May 7 2015, 11:05 PM
DannyH moved this task from Product Review to Done on the Collaboration-Team-Sprint-B-2015-05-20 board.
Content licensed under Creative Commons Attribution-ShareAlike 3.0 (CC-BY-SA) unless otherwise noted; code licensed under GNU General Public License (GPL) or other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL