Page MenuHomePhabricator

Handle MobileWatchlist in regards to Edit Review Improvements
Open, Needs TriagePublic

Description

The T152754: Configure filters in a single extensible place refactoring technically doesn't break this, AFAICT. It doesn't inherit from ChangesListSpecialPage, so it's not affected by whatever we do there.

It has only a few conditions, which it handles its own way (e.g. namespace, hidecategorization).

It also fires SpecialWatchlistQuery, which was already deprecated before this refactoring, and only with an empty array (not with fields provided by the user).

So I don't think the refactoring worsens the situation here, but we should re-evaluate at some point.

Details

Related Gerrit Patches:

Event Timeline

Restricted Application added a project: Collaboration-Team-Triage. · View Herald TranscriptMar 7 2017, 6:00 AM
Restricted Application added a subscriber: Aklapper. · View Herald Transcript
Restricted Application added a project: Growth-Team. · View Herald TranscriptOct 18 2018, 7:02 PM

Change 499864 had a related patch set uploaded (by Jdlrobson; owner: Jdlrobson):
[mediawiki/extensions/MobileFrontend@master] Remove the SpecialMobile(Watchlist|History|Contributions) page and use the core versions

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

Change 499864 abandoned by Jdlrobson:
Remove the SpecialMobile(Watchlist|History|Contributions) page and use the core versions

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