Page MenuHomePhabricator

Surface copyvio as possible issue in Page Curation toolbar "Page info" flyout
Closed, ResolvedPublic

Description

Spun off from T207847

Copyvios was easier to add than the ORES issues, so it was tackled as a separate patch. This task is document that work.

Current behavior:

The Page Curation tool flags Potential issues in the Page info tab of the page curation toolbar, but does not include "Copyvio" as you might see in the feed at Special:NewPagesFeed. Here's an example of such an entry in the feed, see bottom-right:

Screenshot from 2019-07-03 14-07-48.png (177×1 px, 44 KB)

Desired behavior:

"Copyvio" should be included in the list of "Possible issues":

npp info flyout - 2 (790×1 px, 188 KB)

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Actually, there's no way to QA this unless you simulate the copyvio locally by adding the tag manually in the database. See T207847#5297073

Feel free to just move to Product sign-off. This can't be properly tested until it's in production, unfortunately, since the copyvio detection service isn't set up for Beta or even testwiki.

dom_walden added a subscriber: dom_walden.

Actually, there's no way to QA this unless you simulate the copyvio locally by adding the tag manually in the database. See T207847#5297073

Thanks. Doing this I can see that the "Copyvio" issue appears in the Page info flyout. See below.

copyvio_page_info.png (526×660 px, 48 KB)

Feel free to just move to Product sign-off. This can't be properly tested until it's in production, unfortunately, since the copyvio detection service isn't set up for Beta or even testwiki.

It is my understanding that the copyvio service sends an API request to the Pagetriage extension which will add the row you mention above to the database. It would seem to be enough to just simulate as you suggest.

Nevertheless, I will also do a quick check when it is pushed live.

The row in the database tags a particular revision of a page with "Copyvio". The tagged revision does not need to be the latest in order for "Copyvio" to appear in Page info. For example, there might be a delay in the copyvio service or later revisions may be too small for it to consider.

When the database row is removed, "Copyvio" no longer appears in Page info.

Yeah, I'd like to keep this in Dom's column until it has been tested - whether on Beta or in Prod. I believe it went out with the train today.

Yeah, I'd like to keep this in Dom's column until it has been tested - whether on Beta or in Prod. I believe it went out with the train today.

Leon has confirmed that "Copyvio" appears on Page info on en.wiki.

ifried claimed this task.