Page MenuHomePhabricator
Create Task
Maniphest T204455

"copyvio" actions should be publically logged on projects
Closed, ResolvedPublic
Actions

Description

Follow up from https://en.wikipedia.org/wiki/Wikipedia:Bots/Requests_for_approval/EranBot_3#Discussion

This task calls for making a project-specific action that causes a change, however there is no log of these actions as either "contributions" or "logged actions" - for transparency being able to identify the account responsible for changes should be visible to users. Adding an additional Special:Log for this may be the best way to solve this.

Details

SubjectRepoBranchLines +/-
Use potential instead of possible copyright violationmediawiki/extensions/PageTriagemaster+2 -2
Don't publish tag copyvio insert events to Recent Changesmediawiki/extensions/PageTriagemaster+1 -2
Log copyvio insertion eventsmediawiki/extensions/PageTriagemaster+246 -109
Customize query in gerrit

Related Objects
Search...

Event Timeline

Xaosflux created this task.Sep 16 2018, 6:47 PM
JJMC89 edited projects, added PageTriage; removed Epic, Growth-Team (Sprint 0 (Growth Team)).Sep 16 2018, 7:46 PM
Restricted Application added a project: Growth-Team. · View Herald TranscriptSep 16 2018, 7:46 PM
MMiller_WMF moved this task from Inbox to Upcoming Work on the Growth-Team board.Sep 19 2018, 7:12 PM

Thanks, @Xaosflux -- good idea. We'll get to this.

kostajh claimed this task.Sep 20 2018, 4:38 PM
kostajh edited projects, added Growth-Team (Sprint 0 (Growth Team)); removed Growth-Team.
kostajh moved this task from Incoming to In Progress on the Growth-Team (Sprint 0 (Growth Team)) board.
gerritbot subscribed.Sep 24 2018, 3:51 PM

Change 462509 had a related patch set uploaded (by Kosta Harlan; owner: Kosta Harlan):
[mediawiki/extensions/PageTriage@master] Log copyvio insertion events

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

gerritbot added a project: Patch-For-Review.Sep 24 2018, 3:51 PM
kostajh moved this task from In Progress to Code Review on the Growth-Team (Sprint 0 (Growth Team)) board.Sep 24 2018, 4:02 PM
gerritbot added a comment.Sep 25 2018, 2:41 PM

Change 462509 merged by jenkins-bot:
[mediawiki/extensions/PageTriage@master] Log copyvio insertion events

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

SBisson moved this task from Code Review to QA on the Growth-Team (Sprint 0 (Growth Team)) board.Sep 25 2018, 2:44 PM
ReleaseTaggerBot added a project: MW-1.32-notes (WMF-deploy-2018-10-02 (1.32.0-wmf.24)).Sep 25 2018, 3:00 PM
gerritbot added a comment.Sep 25 2018, 7:48 PM

Change 462803 had a related patch set uploaded (by Kosta Harlan; owner: Kosta Harlan):
[mediawiki/extensions/PageTriage@master] Don't publish tag copyvio insert events to Recent Changes

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

kostajh moved this task from QA to Code Review on the Growth-Team (Sprint 0 (Growth Team)) board.Sep 25 2018, 7:51 PM
kostajh moved this task from Code Review to QA on the Growth-Team (Sprint 0 (Growth Team)) board.Sep 25 2018, 7:59 PM
gerritbot added a comment.Sep 25 2018, 8:07 PM

Change 462803 merged by jenkins-bot:
[mediawiki/extensions/PageTriage@master] Don't publish tag copyvio insert events to Recent Changes

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

Etonkovidova moved this task from QA to Needs PM Review on the Growth-Team (Sprint 0 (Growth Team)) board.Sep 27 2018, 6:58 PM
Etonkovidova mentioned this in T205657: [betalabs] duplicate 'Copyvio event' log entries via API request .Sep 27 2018, 7:12 PM
MMiller_WMF closed this task as Resolved.Oct 1 2018, 6:37 PM
MMiller_WMF moved this task from Needs PM Review to Epics In Progress on the Growth-Team (Sprint 0 (Growth Team)) board.
eranroz mentioned this in T201073: Copyvio: Make Eranbot call PageTriage with copyvio info.Oct 2 2018, 6:19 PM
Etonkovidova added a comment.Oct 3 2018, 5:28 PM

The log is present and all log functionality checked (filtering options, deletion/suppression of log entires) - @MMiller_WMF please take a quick look- https://en.wikipedia.beta.wmflabs.org/wiki/Special:Log?type=pagetriage-copyvio&user=Etonkovidova&page=&wpdate=&tagfilter=:

Screen Shot 2018-09-27 at 11.35.14 AM.png (472×1 px, 97 KB)

MMiller_WMF added a comment.EditedOct 11 2018, 10:06 PM

I'm just noticing now that the word "possible" is used in this log, from @Etonkovidova's screenshot. The New Pages Feed UI uses "potential". I remember we discussed this -- is it easy/worth changing so that the log's language aligns with the UI?

Xaosflux added a comment.Oct 11 2018, 10:40 PM

@MMiller_WMF while that is the 'default' - it can be set per project at MediaWiki:Logentry-pagetriage-copyvio-insert

gerritbot added a comment.Oct 12 2018, 7:44 PM

Change 466952 had a related patch set uploaded (by Sbisson; owner: Sbisson):
[mediawiki/extensions/PageTriage@master] Use potential instead of possible copyright violation

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

gerritbot added a comment.Oct 12 2018, 10:56 PM

Change 466952 merged by jenkins-bot:
[mediawiki/extensions/PageTriage@master] Use potential instead of possible copyright violation

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

ReleaseTaggerBot edited projects, added MW-1.32-notes (WMF-deploy-2018-10-16 (1.32.0-wmf.26)); removed MW-1.32-notes (WMF-deploy-2018-10-02 (1.32.0-wmf.24)).Oct 12 2018, 11:00 PM
dom_walden mentioned this in T228669: Add #pagetriage tag to potential copyright violation log entries.Aug 29 2019, 11:09 AM
DannyS712 moved this task from Backlog to PageCuration & NewPagesFeed on the English-Wikipedia-New-Pages-Patrol board.Dec 2 2019, 9:12 PM
Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and 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