UserInfoCard: Display number of times a temporary account has had its IP revealed
Closed, ResolvedPublic0.5 Estimated Story Points
Actions

Assigned To

Authored By

	kostajh
	Jun 15 2025, 8:19 AM

Project Tags

Referenced Files

	F62353815: image.png
	Jun 16 2025, 7:15 PM

	F62353781: image.png
	Jun 16 2025, 7:12 PM

	F62353727: image.png
	Jun 16 2025, 7:09 PM

Subscribers

Description

Summary

If the logged-in user viewing the UserInfoCard has temporary account IP reveal rights, then the UserInfoCard should show how many times a given temporary account has had its IP revealed.

Technical notes

Add output to the CheckUserInfoCardService that includes the count of IP reveals for a given temporary account.

Acceptance criteria

The CheckUserInfoCardService and API endpoint have a count of temporary account IP reveals for users with the temporary account IP reveal permissions
The UserInfoCard displays this data point for users with temporary account IP reveal permissions
make the link conditional on the user having the checkuser-temporary-account-log right

Details

Related Changes in Gerrit:

	Subject	Repo	Branch	Lines +/-
	UserInfoCard: Show how many times a temp account has had its IP revealed	mediawiki/extensions/CheckUser	master	+655 -47

Customize query in gerrit

Related Objects
Search...

		Status	Subtype	Assigned	Task
		Resolved		KColeman-WMF	T384725 [Epic] WE4.2.18 UserInfoCard experiment for assisting patrollers and moderators
		Resolved		• hector.arroyo	T396945 UserInfoCard: Display number of times a temporary account has had its IP revealed

Event Timeline

kostajh created this task.Jun 15 2025, 8:19 AM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJun 15 2025, 8:19 AM

@KColeman-WMF @Niharika I filed this task based on T395212#10915463. What do you think?

Johannnes89 subscribed.Jun 15 2025, 8:59 AM

In T396945#10915676, @kostajh wrote:

@KColeman-WMF @Niharika I filed this task based on T395212#10915463. What do you think?

I think it sounds fine. @MMoss_WMF any thoughts on this idea?

@Johannnes89 would you also want to have "Date of last IP reveal" or is the count of IP reveals enough to be a useful signal to you?

I think the date is less important than knowing if someone has already checked the temporary account or not, but if it’s not much effort to implement… (I might be the wrong person to judge this, as I have access to Special:Log/checkuser-temporary-account anyway). I‘m assuming patrollers and admins would be happy about the date, because recency of the last check might be another reason (not) to check a temporary account's IP address.

In T396945#10916431, @Niharika wrote:

In T396945#10915676, @kostajh wrote:

@KColeman-WMF @Niharika I filed this task based on T395212#10915463. What do you think?

I think it sounds fine. @MMoss_WMF any thoughts on this idea?

The idea looks good to me, no further thoughts right now.

We could add a line similar to this:

IP reveals: 1 (last checked 22 Feb 2025)

Would the text link anywhere? There may be better ways to word this, but we also want to keep the text concise and not too long for translations.

KColeman-WMF attached a referenced file: F62353815: image.png. (Show Details)Jun 16 2025, 7:15 PM

In T396945#10920129, @KColeman-WMF wrote:

We could add a line similar to this:

IP reveals: 1 (last checked 22 Feb 2025)

Would the text link anywhere? There may be better ways to word this, but we also want to keep the text concise and not too long for translations.

It can link to Special:Log?type=checkuser-temporary-account&page=User:{username}

IP reveals sounds fine to me.

In T396945#10920163, @kostajh wrote:

In T396945#10920129, @KColeman-WMF wrote:

Would the text link anywhere?

It can link to Special:Log?type=checkuser-temporary-account&page=User:{username}

Admins & temporary account viewers cannot open the log, not sure if it's good UX to link something that only few people (CU, stewards, ombuds, staff) can access? If it's not too much work and the link can be provided just for those groups it would save us some clicks though...

In T396945#10920210, @Johannnes89 wrote:

In T396945#10920163, @kostajh wrote:

In T396945#10920129, @KColeman-WMF wrote:

Would the text link anywhere?

It can link to Special:Log?type=checkuser-temporary-account&page=User:{username}

Admins & temporary account viewers cannot open the log, not sure if it's good UX to link something that only few people (CU, stewards, ombuds, staff) can access? If it's not too much work and the link can be provided just for those groups it would save us some clicks though...

Yes, we can make the link conditional on the user having the checkuser-temporary-account-log right. As this feature is all about providing convenience to the end-user, it seems like a good idea to provide the link when it can be used.

kostajh added a project: Trust and Safety Product Sprint (Sprint Baklava (June 16 - July 4)).Jun 19 2025, 7:17 AM

• hector.arroyo changed the task status from Open to In Progress.Jun 27 2025, 12:38 PM

• hector.arroyo claimed this task.

• hector.arroyo moved this task from Priority Backlog to In Progress on the Trust and Safety Product Sprint (Sprint Baklava (June 16 - July 4)) board.

kostajh updated the task description. (Show Details)Jun 27 2025, 1:16 PM

Change #1164992 had a related patch set uploaded (by Harroyo-wmf; author: Harroyo-wmf):

[mediawiki/extensions/CheckUser@master] UserInfo: Display number of times a temporary account has had its IP revealed

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

gerritbot added a project: Patch-For-Review.Jun 30 2025, 12:13 PM

kostajh moved this task from In Progress to Needs Review on the Trust and Safety Product Sprint (Sprint Baklava (June 16 - July 4)) board.Jul 2 2025, 10:28 AM

Pending to make the frontend changes to show the new value in the info card

• hector.arroyo moved this task from Needs Review to In Progress on the Trust and Safety Product Sprint (Sprint Baklava (June 16 - July 4)) board.Jul 2 2025, 12:25 PM

kostajh moved this task from Inbox to Nice-to-have for MVP rollout on the CheckUser-UserInfoCard board.Jul 3 2025, 12:06 PM

Pending to update frontend tests

kostajh edited projects, added: Trust and Safety Product Sprint (Sprint Cannoli (July 7 - July 25)); removed: Trust and Safety Product Sprint (Sprint Baklava (June 16 - July 4)).Jul 7 2025, 9:45 AM

kostajh moved this task from Priority Backlog to In Progress on the Trust and Safety Product Sprint (Sprint Cannoli (July 7 - July 25)) board.

• hector.arroyo moved this task from In Progress to Needs Review on the Trust and Safety Product Sprint (Sprint Cannoli (July 7 - July 25)) board.Jul 7 2025, 2:09 PM

We'll come back to this when we have more time for review and QA.

KColeman-WMF attached a referenced file: F62353781: image.png. (Show Details)Jul 18 2025, 6:48 PM

kostajh moved this task from Nice-to-have for MVP rollout to Tasks for sprint 2, can happen after August 4 on the CheckUser-UserInfoCard board.Jul 23 2025, 10:26 AM

kostajh set the point value for this task to 0.5.Jul 23 2025, 10:29 AM

kostajh added a project: Trust and Safety Product Sprint (Sprint Rum baba (July 28 - August 15)).Jul 25 2025, 12:26 PM

kostajh moved this task from Priority Backlog to Needs Review on the Trust and Safety Product Sprint (Sprint Rum baba (July 28 - August 15)) board.

~~I've rebased the previous patch https://gerrit.wikimedia.org/r/c/mediawiki/extensions/CheckUser/+/1164992 and is now ready for review again.~~ Tests are failing after the rebase, I've drafted the patch.