Notifications - fix pager
ClosedPublic
Actions

Authored by btrahan on Aug 27 2014, 7:14 PM.

Details

Reviewers

epriestley

Maniphest Tasks

T5975: Next button in notifications page doesn't work

Commits

Restricted Diffusion Commit
rP85b767bbdc65: Notifications - fix pager

Summary

D10281 upgraded us to modern infrastructure but I think forget to set this little helper to return true. Fixes T5975.

Test Plan

paged through notifications with glee

Diff Detail

Repository

rP Phabricator

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

btrahan updated this revision to Diff 24956.Aug 27 2014, 7:14 PM

btrahan retitled this revision from to Notifications - fix pager.

btrahan updated this object.

btrahan edited the test plan for this revision. (Show Details)

btrahan added a reviewer: epriestley.

btrahan added a task: T5975: Next button in notifications page doesn't work.

Herald added subscribers: Korvin, epriestley. · View Herald TranscriptAug 27 2014, 7:14 PM

Harbormaster completed remote builds in B2368: Diff 24956.Aug 27 2014, 7:15 PM

I think we could cursor page this on chronologicalKey but this is reasonable, at least until we have a better reason to support cursor paging.

This revision is now accepted and ready to land.Aug 27 2014, 7:16 PM

yeah, I tried to get that going actually and it was being all funky on me. :/

Closed by commit rP85b767bbdc65 (authored by @btrahan).

Revision Contents
Changeset List

Path

Size

src/

applications/

notification/

query/

PhabricatorNotificationSearchEngine.php

4 lines

Diff 24958

View Options

src/applications/notification/query/PhabricatorNotificationSearchEngine.php

Notifications - fix pagerClosedPublicActions