I'm leaning towards decline on this. The option was originally removed in 2012 in order to streamline the reviewing process and match the behavior of Twinkle. I'm not convinced there is enough of a need for it to justify re-adding it, especially since the interface is already quite complicated as it is.

@kaldari Twinkle now includes a checkbox to disable patrolling of the article. There are many situations in which it is inappropriate to mark a page as patrolled when you are tagging it.

There are many situations in which it is inappropriate to mark a page as patrolled when you are tagging it.

@TheDragonFire - Can you elaborate on that? When we were originally developing the Page Curation tools we were asked to make tagging automatically mark the article as reviewed (to save the reviewers a step). In what situations would a reviewer tag an article, but not want to mark it as reviewed?

Moving this to 'Triaged but future' because many Page Curation tasks are currently part of the 2019 Wishlist Survey

Perhaps the default case should be not marking it as patrolled? I.e., rather than on option not to patrol the page, we simply change it so that, rather than automatically patrolling the page, we don't patrol the page at all unless the "mark this page as patrolled" option in activated

In T148353#2724201, @kaldari wrote:

I'm leaning towards decline on this. The option was originally removed in 2012 in order to streamline the reviewing process and match the behavior of Twinkle. I'm not convinced there is enough of a need for it to justify re-adding it, especially since the interface is already quite complicated as it is.

as noted in T148353#3451322, Twinkle has this option now. If the default is to mark it as patrolled (T148353#5356026), then it isn't much of a throttle on the reviewing process, but allows users to add tags without needing to review the entire article

Change 530648 had a related patch set uploaded (by DannyS712; owner: DannyS712):
[mediawiki/extensions/PageTriage@master] Partially revert "bug39208 - Add auto review to tagging/deletion"

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

Change 530649 had a related patch set uploaded (by DannyS712; owner: DannyS712):
[mediawiki/extensions/PageTriage@master] Partially revert "bug39208 - Add auto review to tagging/deletion"

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

Change 530648 abandoned by DannyS712:
Partially revert "bug39208 - Add auto review to tagging/deletion"

Reason:
In favor of Icf5d1c73f3ac09c0530a20ca3fa86a4a618aea0a

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

Should I move this to external in Growth-Team ? The patch is ready for review

Moving to inbox to get more eyes - can someone take a look at the patch? Automatically patrolling when tagging was activated based on T41208#468859, and the NPP community would like to have the ability to again choose not to patrol pages when tagging them. See https://en.wikipedia.org/wiki/Wikipedia_talk:New_pages_patrol/Reviewers#Decoupling_maintenance_tagging_and_reviewing for current discussion

As @DannyS712 has pointed out this has received considerable support from among the NPP Community.

@MMiller_WMF @JTannerWMF @Catrope @kostajh @Trizek-WMF @Tgr can someone please take a look at the patch provided? The NPP has consensus to restore this feature that was never requested for removal. (List based on https://phabricator.wikimedia.org/project/members/1114/) Thanks

@DannyS712 -- thank you for working on this. I recently caught up with the members of the Community Tech team, and it sounds like they left you some comments in your code for you to work on. Is that the current status?

In T148353#5661278, @MMiller_WMF wrote:

@DannyS712 -- thank you for working on this. I recently caught up with the members of the Community Tech team, and it sounds like they left you some comments in your code for you to work on. Is that the current status?

I followed up on the feedback left, and the current status is waiting for further review. Thanks

@MMiller_WMF any updates?

I've reviewed the pending patch at https://gerrit.wikimedia.org/r/c/mediawiki/extensions/PageTriage/+/530649

@DannyS712 you've answered some of my concerns well, but the two times that I actually tested the patch, it didn't run. The first time it crashed (you've fixed the crashes that I pointed out) and the second it simply does not appear to run properly and I did not find the behavior change you've set.

I might be missing something that is actually running, so I need your confirmation that you have run this locally, successfully, before continuing with the review.

In T148353#5776330, @Mooeypoo wrote:

I've reviewed the pending patch at https://gerrit.wikimedia.org/r/c/mediawiki/extensions/PageTriage/+/530649

@DannyS712 you've answered some of my concerns well, but the two times that I actually tested the patch, it didn't run. The first time it crashed (you've fixed the crashes that I pointed out) and the second it simply does not appear to run properly and I did not find the behavior change you've set.

I might be missing something that is actually running, so I need your confirmation that you have run this locally, successfully, before continuing with the

I had to figure out how to install vagrant, mediawiki, etc. - the issue was that it was there, but hidden. I've uploaded a new patch that I have indeed tested locally (first time I was able to test things locally, yay)!

That's great!! Let us know if you need any help on those (though it looks like you're super on top of everything :)

I've looked at the patch again -- it looks good to me. I also like the fact we don't need to put on an event listener anymore (these create some annoyances in PageTriage with the potential to be attached more than once)

However, In a twist of bitter irony (and cruel cruel fate) my vagrant has crashed, and my internet has been acting up very badly, and I couldn't manage to actually retest the patch for merging. So I've +1'ed the patch, and I'll be asking someone from the team to double-check it locally and merge.

Thank you for the work, @DannyS712

Change 530649 merged by jenkins-bot:
[mediawiki/extensions/PageTriage@master] Partially revert "bug39208 - Add auto review to tagging/deletion"

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