Page MenuHomePhabricator

Remove GuidedTour from Special:Investigate
Closed, ResolvedPublic1 Estimated Story Points

Assigned To
Authored By
Niharika
Oct 1 2020, 8:00 PM
Referenced Files
F32415848: image.png
Oct 27 2020, 1:14 AM
F32415847: image.png
Oct 27 2020, 1:14 AM
F32415841: image.png
Oct 27 2020, 1:07 AM
F32415843: image.png
Oct 27 2020, 1:07 AM
F32415809: image.png
Oct 26 2020, 11:59 PM
F32415800: image.png
Oct 26 2020, 11:59 PM
F32415796: image.png
Oct 26 2020, 11:59 PM
F32415811: image.png
Oct 26 2020, 11:59 PM

Description

Goal

Following our discussion in T263540: Consolidate CheckUser front-end modules we should remove the GuidedTour feature once it has outlived its utility.

Questions:

Can we get any metrics about how many times:

  • GT popups were displayed?
  • users clicked through the GT popups?

These are addressed in T265106.

Acceptance criteria

Event Timeline

Niharika triaged this task as Medium priority.

Can we get any metrics

This should be recorded under Schema:GuidedTour. Query for the tourName of checkuserinvestigateform (the initial form page) and checkuserinvestigate (after starting an investigation).

Can we get any metrics

This should be recorded under Schema:GuidedTour. Query for the tourName of checkuserinvestigateform (the initial form page) and checkuserinvestigate (after starting an investigation).

Oh thanks @dbarratt. @jwang Flagging this for you. Let's talk about this in our next 1:1.

Can we get any metrics

This should be recorded under Schema:GuidedTour. Query for the tourName of checkuserinvestigateform (the initial form page) and checkuserinvestigate (after starting an investigation).

Quick correction - @jwang and I found out that that schema is actually inactive (see talk page) and there are 6 new schemas instead of it. One of which is https://meta.wikimedia.org/wiki/Schema:GuidedTourGuiderImpression

jwang updated the task description. (Show Details)
jwang updated the task description. (Show Details)
This comment was removed by jwang.

@Niharika is this ticket still needed? Could it be closed now?

@Niharika should we remove GuidedTour integration with Special:Investigate?

@Niharika should we remove GuidedTour integration with Special:Investigate?

I would advocate removing the tour, as it is now fully deployed on all wikis and has been enabled by default on third-party wikis for a while.

It will also allow removing some resourceloader modules, making the CSS/JS content load of CheckUser smaller.

Yes I think it's safe to remove it now.

Per T264359#10159362 we should work on dropping the guided tour. Doing this will:

  • Reduce code complication
  • Remove 2 ResourceLoader modules (the total number of modules on WMF wikis has been raised as an issue)
Dreamy_Jazz set the point value for this task to 1.Aug 11 2025, 2:11 PM

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

[mediawiki/extensions/CheckUser@master] Remove guided tour for Special:Investigate

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

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

[integration/config@master] Remove dependency on GuidedTour from CheckUser

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

Change #1187434 merged by jenkins-bot:

[mediawiki/extensions/CheckUser@master] Remove guided tour for Special:Investigate

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

The GuidedTour part has been removed, so AFAICS it just means updating the CheckUser documentation on mediawiki.org (so moving back to 'In Progress' for that)

https://gerrit.wikimedia.org/r/c/integration/config/+/1187443 has received a +1 from me and requires merging by someone outside the team

Preferences for checkuser-investigate-form-tour-seen and checkuser-investigate-tour-seen were deleted via maintenance scripts on all wikis.

Moving this to Done - we can close it once https://gerrit.wikimedia.org/r/c/integration/config/+/1187443 is merged (another reviewer was added).

Change #1189885 had a related patch set uploaded (by Dreamy Jazz; author: Tchanders):

[mediawiki/extensions/CheckUser@REL1_44] Remove guided tour for Special:Investigate

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

Change #1189885 abandoned by Dreamy Jazz:

[mediawiki/extensions/CheckUser@REL1_44] Remove guided tour for Special:Investigate

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

Change #1187443 merged by jenkins-bot:

[integration/config@master] Zuul: [mediawiki/extensions/CheckUser] Remove dependency on GuidedTour

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

Mentioned in SAL (#wikimedia-releng) [2025-09-22T10:40:08Z] <James_F> Zuul: [mediawiki/extensions/CheckUser] Remove dependency on GuidedTour, for T264359

Thanks for merging the CI config change. Marking this as resolved