Page MenuHomePhabricator

Autopatrolled users should be able to unpatrol their own articles
Closed, ResolvedPublicFeature

Description

Requested by @TechnoSquirrel69 at https://en.wikipedia.org/wiki/Wikipedia_talk:WikiProject_Articles_for_creation#Observations

Feature summary (what you would like to be able to do and where):

  • Autopatrolled users should be able to unpatrol their own articles.
  • Suggest doing this by adding an "Add to the New Pages Feed" link to the left "tools" menu, similar to how articles that have been reviewed for more than 30 days also get this link. This lets us reuse the i18n message, and avoids having to display the NPP toolbar (since the autopatrolled user might not be an NPP)

Use case(s) (list the steps that you performed to discover that problem, and describe the actual underlying problem which you want to solve. Do not describe only a solution):

  • Autopatrolled users that aren't NPPs that don't feel 100% sure about the notability of an article
  • Autopatrolled users that are AFC reviewers and aren't NPPs that don't feel 100% sure about the notability of a draft they're accepting

Benefits (why should this be implemented?):

  • Second set of eyes on questionable articles

Event Timeline

Autopatrolled users should be able to unpatrol their own articles.

This won't be true for AFC drafts, it will be somebody else's article (atleast from PageTriage's POV) :(

I feel like this would need wider community consensus before being implemented since implementing this will basically require giving autopatrolled users the ability to unpatrol basically any page in the NPP queue (or atleast all article that have previously moved from draftspace into the NPP queue).

Good point. I didn't think of that. At a minimum that complicates the logic (we would have to store who did a move that created an autopatrolled status), or we'd need to reduce the scope of this ticket to just own articles.

I agree that a non-npp being able to unpatrol everything and anything is too much power.

That's a bummer. I suppose this would mean the feature wouldn't be applicable to AfC reviews, but maybe it would still be beneficial to allow editors to unpatrol "their own" articles for other circumstances. As Novem Linguae mentioned at WT:AFC, this smaller change may be uncontroversial enough to not require a community discussion.

That's a bummer. I suppose this would mean the feature wouldn't be applicable to AfC reviews, but maybe it would still be beneficial to allow editors to unpatrol "their own" articles for other circumstances. As Novem Linguae mentioned at WT:AFC, this smaller change may be uncontroversial enough to not require a community discussion.

Unreviewing ones own article seems like it should be fine :)

Change 977284 had a related patch set uploaded (by Sohom Datta; author: Sohom Datta):

[mediawiki/extensions/PageTriage@master] Allow autopatrolled users to unreview their own pages

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

MPGuy2824 changed the task status from Open to In Progress.Dec 1 2023, 3:25 AM
MPGuy2824 assigned this task to Soda.
MPGuy2824 moved this task from Priority small features to Code Review on the PageTriage board.

Test wiki created on Patch demo by SCardenas (WMF) using patch(es) linked to this task:
https://patchdemo.wmflabs.org/wikis/ce52fe915a/w

Scardenasmolinar subscribed.

I set up a Patch Demo, and get this error whenever I click on the Add to the New Pages Feed link

Screenshot 2023-12-12 at 12.17.13.png (118×1 px, 72 KB)

Test wiki created on Patch demo by Sohom Datta using patch(es) linked to this task:
https://patchdemo.wmflabs.org/wikis/4b5bcbc889/w

Test wiki on Patch demo by SCardenas (WMF) using patch(es) linked to this task was deleted:

https://patchdemo.wmflabs.org/wikis/ce52fe915a/w/

Test wiki created on Patch demo by SCardenas (WMF) using patch(es) linked to this task:
https://patchdemo.wmflabs.org/wikis/7120a34470/w

Test wiki on Patch demo by SCardenas (WMF) using patch(es) linked to this task was deleted:

https://patchdemo.wmflabs.org/wikis/7120a34470/w/

Change #977284 merged by jenkins-bot:

[mediawiki/extensions/PageTriage@master] Allow autopatrolled users to unreview their own pages

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

Test wiki on Patch demo by Sohom Datta using patch(es) linked to this task was deleted:

https://patchdemo.wmflabs.org/wikis/4b5bcbc889/w/