Page MenuHomePhabricator

Make IPInfo tool available on Special:RecentChanges [M]
Closed, ResolvedPublic

Description

Following T260604, the IPInfo tool is available on history pages, using the revision API endpoint.

Following T263634, the IPInfo tool is available on history pages, using the log API endpoint.

Make the IPInfo tool available on Special:RecentChanges, which will use both endpoints.

Event Timeline

ARamirez_WMF renamed this task from Make IPInfo tool available on Special:RecentChanges to Make IPInfo tool available on Special:RecentChanges [M].Sep 29 2021, 4:23 PM

Change 725373 had a related patch set uploaded (by TsepoThoabala; author: TsepoThoabala):

[mediawiki/extensions/IPInfo@master] Make IPInfo tool available on Special:RecentChanges

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

Change 725373 merged by jenkins-bot:

[mediawiki/extensions/IPInfo@master] Make IPInfo tool available on Special:RecentChanges

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

I noticed two things during review that I didn't feel should block merging @TThoabala's change but should be considered part of this task:

  1. @Prtksxna @Niharika: Special:RecentChanges groups multiple edits in quick succession to the same page and displays them as a single collapsed line item with the user's name still visible. Should IP Info's (i) button be present for this line?

    Further, if you expand the collapsed line item, then each line shows IP Info's (i) button. Is this correct?
CollapsedExpanded
multiple-edits-collapsed.png (1×2 px, 441 KB)
multiple-edits-expanded.png (1×2 px, 457 KB)
  1. Special:RecentChanges polls for new changes every second. You can either manually update the page when new changes are detected or enable the live update feature. In either case, however, when the page is updated, all of IP Info's (i) buttons disappear:

live-updated.png (1×2 px, 483 KB)

  1. @Prtksxna @Niharika: Special:RecentChanges groups multiple edits in quick succession to the same page and displays them as a single collapsed line item with the user's name still visible. Should IP Info's (i) button be present for this line?

Yep, it should. I've created T293332: IP Info icon should be present on collapsed lines on RecentChanges [M]

Further, if you expand the collapsed line item, then each line shows IP Info's (i) button. Is this correct?

Yes, this is correct.

  1. Special:RecentChanges polls for new changes every second. You can either manually update the page when new changes are detected or enable the live update feature. In either case, however, when the page is updated, all of IP Info's (i) buttons disappear:

Oh no, this shouldn't be happening. Created T293333: IP Info icon shouldn't disappear after Recent Changes has loaded new changes [M].

Thanks for all the details and screenshots @phuedx