Page MenuHomePhabricator

IP auto-reveal: Indicate on sidebar tool link if auto-reveal mode is on
Closed, ResolvedPublic

Description

Background

Users need a visual indicator of whether IP auto-reveal mode is on. For the minimal first release, we will change the wording of this sidebar link, to indicate when the mode is on.

User story

As a patroller using IP auto-reveal to view IP addresses of temporary accounts automatically, I want to know whether I have auto-reveal switched on or off.

Specification

In the sidebar tools menu, display the state of IP auto-reveal mode in parentheses, e.g.:

image.png (428×188 px, 16 KB)

Note that we also have designs to do this via a bottom-dock menu, which may replace this in the future: T386496: IP auto-reveal: Allow managing auto-reveal mode from the bottom dock menu.

Technical notes

SidebarLinksHandler::addIPAutoReveal will need to check if IP auto-reveal is on.

REST API handlers currently do this via TemporaryAccountAutoRevealTrait. This trait could be replaced by (or updated to call) a service, for re-use by the hook handler.

Acceptance criteria

  • The text of the sidebar tool link indicates whether IP auto-reveal mode is on or off

Event Timeline

@KColeman-WMF Please feel free to update the task description.

What do you think the link should display when auto-reveal is off? I'm wondering whether it should just say IP auto-reveal or IP auto-reveal (off). It might look a bit strange to explicitly say that it's off, especially for a user who rarely uses the tool.

Yes, I agree it will look strange to explicitly say it's off.

I'm happy to use IP Auto-reveal for the default/off state and IP Auto-reveal (on) for when the tool is enabled.

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

[mediawiki/extensions/CheckUser@master] IP auto-reveal: Move TemporaryAccountAutoRevealTrait to a service

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

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

[mediawiki/extensions/CheckUser@master] IP auto-reveal: Update the tool link to indicate status

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

Change #1147770 merged by jenkins-bot:

[mediawiki/extensions/CheckUser@master] IP auto-reveal: Move TemporaryAccountAutoRevealTrait to a service

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

Change #1148396 merged by jenkins-bot:

[mediawiki/extensions/CheckUser@master] IP auto-reveal: Update the tool link to indicate status

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

Djackson-ctr subscribed.

QA is completed, I have verified the new code changes have been implemented and is functioning as expected (Per the Acceptance Criteria):

When IP Auto-reveal is ON: the sidebar tool link will display IP Auto-reveal (on)

image.png (326×184 px, 6 KB)


When IP Auto-reveal is OFF: the sidebar tool link will display IP Auto-reveal

image.png (303×184 px, 5 KB)