Page MenuHomePhabricator

[betalabs-Regression] Special:Notifications - the pagination does not follow default pattern of 50 messages per page
Closed, ResolvedPublic

Description

Betalabs 1.28.0-alpha (420c520)

  1. Login as a user with more than one page messages - i.e. >50
  2. Go to Special:Notifications page.

User ET8 has 99+ messages.
The first page displays in pagination 1-43

Screen Shot 2016-07-27 at 11.09.32 AM.png (491×1 px, 106 KB)

The second page displays 51-73
Screen Shot 2016-07-27 at 11.09.45 AM.png (488×1 px, 112 KB)

The screen recording shows a different case - the pagination gets broken on the third page.

Event Timeline

Change 301382 had a related patch set uploaded (by Sbisson):
[WIP] Moderate notifications

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

This is caused by the way we filter unrenderable events. This patch (https://gerrit.wikimedia.org/r/301382) tries to fix it by moderating events that cannot be rendered so they are filtered out at the db level on subsequent fetches.

Change 301382 merged by jenkins-bot:
Moderate notifications

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

Checked in betalabs