Page MenuHomePhabricator

Add global contributions count to IPInfo infobox
Closed, ResolvedPublic

Description

Splitting this from T375465: Link to Special:GlobalContributions page from IP Info, as it requires additional product/design/maybe legal work.

User story

As an editor investigating an IP for cross-wiki abuse, I'd like to easily find and view global contributions for an IP address, in order to assess and mitigate cross-wiki vandalism.

Design

gc-link.png (1,087×1,188 px, 106 KB)

Specifications
Technical Notes

Event Timeline

Change #1118826 had a related patch set uploaded (by STran; author: STran):

[mediawiki/extensions/IPInfo@master] [WIP] Add global contribution count to infobox widget

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

@STran Thanks for filing this.

I believe we're not expecting this to block major pilots, and are just filing it in case we hear that it is very important to users. If it is, we will open a conversation with Legal about possibly showing this count to users without IP reveal permissions, since the permissions check is too expensive.

Note that the global edits count has been requested during a feedback round for Account reputation designs:

"For the steward workflow, global edit count is probably the most important metric."

We have the go-ahead from legal to display the number of global edits by temporary accounts from an IP address to IPInfo users, without checking IP reveal permissions at each wiki.

Change #1136693 had a related patch set uploaded (by STran; author: STran):

[mediawiki/extensions/IPInfo@master] [WIP] Add a hook to IPInfoHandler to allow extensions to add data to API results

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

Change #1136694 had a related patch set uploaded (by STran; author: STran):

[mediawiki/extensions/CheckUser@master] [WIP] Add global contributions count to IPInfo infobox

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

Change #1118826 abandoned by STran:

[mediawiki/extensions/IPInfo@master] [WIP] Add global contribution count to infobox widget

Reason:

Prefer Id8ebc29bd1af09cc0f14dfcb4942a7cae8facb6b

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

Change #1118824 had a related patch set uploaded (by STran; author: STran):

[mediawiki/extensions/CheckUser@master] Add CheckUserGlobalContributionsLookup service

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

Change #1118825 had a related patch set uploaded (by STran; author: STran):

[mediawiki/extensions/CheckUser@master] Expose global contribution count

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

Change #1138074 had a related patch set uploaded (by STran; author: STran):

[integration/config@master] Zuul: [mediawiki/extensions/CheckUser] Add IPInfo phan dependency

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

Change #1138074 merged by jenkins-bot:

[integration/config@master] Zuul: [mediawiki/extensions/CheckUser] Add IPInfo phan dependency

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

Change #1136693 merged by jenkins-bot:

[mediawiki/extensions/IPInfo@master] Add a hook to IPInfoHandler to allow extensions to add data to API results

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

Change #1118824 merged by jenkins-bot:

[mediawiki/extensions/CheckUser@master] Add CheckUserGlobalContributionsLookup service

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

Change #1118825 merged by jenkins-bot:

[mediawiki/extensions/CheckUser@master] Expose global contribution count through `CheckUserGlobalContributionsLookup`

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

Change #1136694 merged by jenkins-bot:

[mediawiki/extensions/CheckUser@master] Add global contributions count to IPInfo infobox

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

Surely the message checkuser-ipinfo-global-contributions-value should be accommodated for a single global edit or zero global edits like ipinfo-value-local-edits is? Now it would display "1 global edits".

Change #1146531 had a related patch set uploaded (by Tchanders; author: Tchanders):

[mediawiki/extensions/CheckUser@master] Handle plurals and singular correctly in message

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

Change #1146531 merged by jenkins-bot:

[mediawiki/extensions/CheckUser@master] Handle plurals and singular correctly in message

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

Change #1146675 had a related patch set uploaded (by STran; author: STran):

[mediawiki/extensions/IPInfo@master] Pass performer as an Authority through the IPInfoHandlerRun hook

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

Change #1146676 had a related patch set uploaded (by STran; author: STran):

[mediawiki/extensions/CheckUser@master] Perfer the authority passed through from IPInfo over the request context's

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

Change #1146881 had a related patch set uploaded (by STran; author: STran):

[mediawiki/extensions/CheckUser@master] Temporary skip test in CI to accomodate a hook signature update

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

Change #1146943 had a related patch set uploaded (by STran; author: STran):

[mediawiki/extensions/CheckUser@master] Revert "Temporarily skip test in CI to accomodate a hook signature update"

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

Change #1146972 had a related patch set uploaded (by STran; author: STran):

[mediawiki/extensions/CheckUser@master] [WIP] Prefer the authority passed through from IPInfo over the request context's

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

Change #1146971 had a related patch set uploaded (by STran; author: STran):

[mediawiki/extensions/CheckUser@master] Temporarily skip test in CI to accomodate a hook signature update

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

Change #1146881 abandoned by STran:

[mediawiki/extensions/CheckUser@master] Temporarily skip test in CI to accomodate a hook signature update

Reason:

Prefer I8dda1db65d923db8b2274c06346da3d602713433

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

Change #1146943 abandoned by STran:

[mediawiki/extensions/CheckUser@master] Revert "Temporarily skip test in CI to accomodate a hook signature update"

Reason:

I8dda1db65d923db8b2274c06346da3d602713433

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

Change #1146676 abandoned by STran:

[mediawiki/extensions/CheckUser@master] Prefer the authority passed through from IPInfo over the request context's

Reason:

I8dda1db65d923db8b2274c06346da3d602713433

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

Change #1146971 merged by jenkins-bot:

[mediawiki/extensions/CheckUser@master] Temporarily skip test in CI to accomodate a hook signature update

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

Change #1146675 merged by jenkins-bot:

[mediawiki/extensions/IPInfo@master] Pass performer as an Authority through the IPInfoHandlerRun hook

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

Change #1146972 merged by jenkins-bot:

[mediawiki/extensions/CheckUser@master] Prefer the authority passed through from IPInfo over the request context's

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

Djackson-ctr subscribed.

QA is completed, I have verified the new code has been implemented and is functioning as expected (Global Contributions count has been added to the IPInfo infobox).