Edit patrol - Flow issues (tracking)
Open, MediumPublic
Actions

Assigned To

None

Authored By

	Quiddity
	Sep 30 2014, 1:27 AM

Description

If the preference "Group changes by page in recent changes and watchlist" is enabled, then all Flow edits get the red exclamation mark indicating "unpatrolled".

(See screenshot, where Quiddity's edit to the "Loveless" topic should have been marked as patrolled, as it was in the Project:Sandbox)

If that preference is turned off, then no Flow edits get marked as needing patrol. (All edits by editors who are not in a group that is "autopatrolled", should be marked as needing patrol, if a wiki is configured to use that.)

See mw:Help:Patrolled edits for details.

To know whether a page should show the 'Mark as patrolled' link, we need to store a mapping between the Flow revision and the RC table (the RC table currently points to the Flow revision, but not in a way that can be queried).

(Forked from T72513 - On trello at https://trello.com/c/k8CJgfBd/)

Version: unspecified
Severity: normal
Attached screenshot:

www.mediawiki.org_2014-09-29_18-18-21.png (357×776 px, 98 KB)

Details

Reference: bz71456

Related Objects
Search...

Status	Subtype	Assigned	Task
Open		None	T85540 Feed (watchlist/RC/contribs) - Flow issues (tracking)
Open		None	T73456 Edit patrol - Flow issues (tracking)
Resolved		Catrope	T94451 A3. Implement auto-patrolling for Flow
Resolved		SBisson	T94452 C2. Implement patrolling other people's edits in Flow
Resolved		SBisson	T94453 C3. Fix RC patrolling rendering for Flow (works in grouped RC, broken in regular RC)
Resolved		SBisson	T95025 C4. Fix watchlist patrolling rendering for Flow in regular watchlist (works in grouped watchlist)
Open		None	T102191 Patrol Log displays numerous "marked revision 0 of page " entries
Open	BUG REPORT	None	T58989 User talk edit by autopatrol user not autopatrolled
Open		None	T104083 Include Flow revisions in RecentChanges API
Open		None	T102310 'Mark this page as patrolled' message should be more specific
Duplicate		Etonkovidova	T114859 Impossible to mark a new topic as patrolled
Duplicate		None	T128379 Session failure when you want to "mark as patrolled" a diff on Flow
Open		None	T96449 Deleting or suppressing a Flow topic should mark the topic and the posts as deleted/suppressed
Open		None	T115043 Patrolling: Discrepancies in displaying 'Mark this page as patrolled' option
Resolved		SBisson	T133341 Flow "mark as patrolled" links don't work (claims "Session failure")
Duplicate		None	T135928 Flow doesn't support revision patroling
Open		None	T95549 Unable to undo multiple damaging actions in Flow discussions
Open		None	T93494 Rollback functionality is needed in Flow

Event Timeline

• bzimport raised the priority of this task from to Needs Triage.Nov 22 2014, 3:50 AM

• bzimport added a project: StructuredDiscussions.

• bzimport set Reference to bz71456.

• bzimport added a subscriber: Unknown Object (MLST).

Quiddity created this task.Sep 30 2014, 1:27 AM

in backlog: https://trello.com/c/prYEn0Yw

Quiddity removed a subscriber: Maryana.Dec 19 2014, 1:41 AM

• Mattflaschen-WMF added a parent task: T72513: R2. Bad Flow link targets in usenewrc (Enhanced RC) in watchlist and recentchanges (Portuguese).Feb 2 2015, 7:33 PM

• Mattflaschen-WMF mentioned this in T72513: R2. Bad Flow link targets in usenewrc (Enhanced RC) in watchlist and recentchanges (Portuguese).

• DannyH added a project: Collaboration-Team-Triage.Feb 2 2015, 7:34 PM

• DannyH set Security to None.

• Mattflaschen-WMF removed a parent task: T72513: R2. Bad Flow link targets in usenewrc (Enhanced RC) in watchlist and recentchanges (Portuguese).Feb 2 2015, 7:39 PM

• DannyH triaged this task as Medium priority.Feb 3 2015, 6:47 PM

In the English Wikipedia RC patrol is not used, but it is used a lot in the Hebrew Wikipedia and in many other projects (go to http://noc.wikimedia.org/conf/highlight.php?file=InitialiseSettings.php and search for "wgUseRCPatrol").

People are complaining about it that edits to Flow board that should appear as auto-patrolled are not, so this is becoming important.

Ah, I came to file a bug about this and I'm happy (sort of) ;) that it already exists. This is a problem in ca.wiki as well:

Screenshot_from_2015-03-27_09:43:28.png (900×1 px, 239 KB)

Two of the three "!" in the screenshot are false shots.

Patrolling is important in ca.wiki, and I bet in small wikis in general, where you cannot assume that 'someone else has watched that diff before me'. Seeing the "!" in my watchlist or not, frequently marks a difference between checking that diff right now or maybe later. If the watchlist entry marks only one edit from an editor I know then fine, but luckily we are in a situation where it is possible not to know all the autopatroled users. :)

Finding that there is nothing to patrol is between slightly annoying (like Peter crying again 'the wolf!') and deceptive (in small wikis you get excited when newcomers / occasional editors are commenting in discussion pages).

Cool, thanks for bringing it up... We'll talk about it for the next sprint.

• DannyH moved this task from Team discussion to Product/Design Work on the Collaboration-Team-Triage board.Mar 30 2015, 6:38 PM

• DannyH moved this task from Product/Design Work to Team discussion on the Collaboration-Team-Triage board.Mar 31 2015, 11:24 PM

• DannyH moved this task from Team discussion to Product/Design Work on the Collaboration-Team-Triage board.Mar 31 2015, 11:45 PM

He7d3r updated the task description. (Show Details)Apr 3 2015, 5:20 PM

Qgil mentioned this in T94451: A3. Implement auto-patrolling for Flow.Apr 6 2015, 10:32 AM

• Spage unsubscribed.Apr 6 2015, 8:35 PM

• Mattflaschen-WMF added a subtask: T95025: C4. Fix watchlist patrolling rendering for Flow in regular watchlist (works in grouped watchlist).Apr 6 2015, 8:48 PM

• Mattflaschen-WMF updated the task description. (Show Details)Apr 15 2015, 7:02 PM

• DannyH closed subtask T94451: A3. Implement auto-patrolling for Flow as Resolved.May 7 2015, 11:05 PM

jeremyb subscribed.May 21 2015, 7:56 PM

• Mattflaschen-WMF renamed this task from Flow: Patrolled edits in Flow not marked properly or consistently to Flow: Patrolled edits in Flow not marked properly or consistently (tracking).Jun 2 2015, 5:53 PM

• DannyH closed subtask T95025: C4. Fix watchlist patrolling rendering for Flow in regular watchlist (works in grouped watchlist) as Resolved.Jun 2 2015, 6:00 PM

• DannyH reopened subtask T95025: C4. Fix watchlist patrolling rendering for Flow in regular watchlist (works in grouped watchlist) as Open.

• DannyH closed subtask T94453: C3. Fix RC patrolling rendering for Flow (works in grouped RC, broken in regular RC) as Resolved.Jun 10 2015, 5:20 PM

• DannyH closed subtask T95025: C4. Fix watchlist patrolling rendering for Flow in regular watchlist (works in grouped watchlist) as Resolved.

• DannyH closed subtask T94452: C2. Implement patrolling other people's edits in Flow as Resolved.Jun 12 2015, 4:38 PM

• Mattflaschen-WMF added a subtask: T102191: Patrol Log displays numerous "marked revision 0 of page " entries.Jun 12 2015, 11:56 PM

• DannyH moved this task from Product/Design Work to Tracking tickets on the Collaboration-Team-Triage board.Jul 22 2015, 9:14 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJul 22 2015, 9:14 PM

• DannyH renamed this task from Flow: Patrolled edits in Flow not marked properly or consistently (tracking) to Edit patrol - Flow issues (tracking).Jul 24 2015, 12:12 AM

• DannyH moved this task from Tracking tickets to Flow Holding Area on the Collaboration-Team-Triage board.Aug 21 2015, 9:29 PM

Quiddity added subtasks: T58989: User talk edit by autopatrol user not autopatrolled, T104083: Include Flow revisions in RecentChanges API, T102310: 'Mark this page as patrolled' message should be more specific.Sep 25 2015, 5:48 PM

Trizek-WMF added a parent task: T114859: Impossible to mark a new topic as patrolled.Oct 7 2015, 8:08 AM

Trizek-WMF removed a parent task: T114859: Impossible to mark a new topic as patrolled.

Trizek-WMF added a subtask: T114859: Impossible to mark a new topic as patrolled.

Quiddity added a subtask: T128379: Session failure when you want to "mark as patrolled" a diff on Flow.Mar 3 2016, 6:07 PM

Qgil unsubscribed.Mar 3 2016, 8:42 PM

Trizek-WMF subscribed.Mar 4 2016, 9:01 AM

Quiddity moved this task from Flow Holding Area to Tracking tickets on the Collaboration-Team-Triage board.Mar 7 2016, 7:04 PM

Quiddity added a project: Tracking-Neverending.Mar 23 2016, 6:19 PM

He7d3r added a subtask: T96449: Deleting or suppressing a Flow topic should mark the topic and the posts as deleted/suppressed.May 7 2016, 6:35 PM

Aklapper added subtasks: T115043: Patrolling: Discrepancies in displaying 'Mark this page as patrolled' option , T133341: Flow "mark as patrolled" links don't work (claims "Session failure"), T135928: Flow doesn't support revision patroling.May 22 2016, 11:23 AM