Limit notifications to latest 10, instead of 15
ClosedPublic
Actions

Authored by chad on Aug 28 2017, 10:12 PM.

Tags

None

Referenced Files

	F15588904: D18486.id44408.diff
	Fri, May 9, 10:12 AM

	F15568481: D18486.id44408.diff
	Sat, May 3, 4:13 AM

	F15550082: D18486.diff
	Sun, Apr 27, 9:13 AM

	F15549449: D18486.id.diff
	Sun, Apr 27, 5:48 AM

	F15546660: D18486.diff
	Sat, Apr 26, 4:22 PM

	F15419297: D18486.diff
	Mar 21 2025, 5:07 AM

	F15411093: D18486.diff
	Mar 19 2025, 8:49 AM

	F15410104: D18486.diff
	Mar 19 2025, 6:47 AM

View All Files

Subscribers

Korvin

Details

Reviewers

epriestley

Commits

rP0609133f4564: Limit notifications to latest 10, instead of 15

Summary

This panel just gets super tall at 15 now that date is on it's own line.

Test Plan

Reload panel, count to 10.

Diff Detail

Repository

rP Phabricator

Branch

notifications-2 (branched from master)

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 18194
Build 24464: Run Core Tests
Build 24463: arc lint + arc unit

Event Timeline

chad created this revision.Aug 28 2017, 10:12 PM

Herald added a subscriber: Korvin. · View Herald TranscriptAug 28 2017, 10:12 PM

epriestley accepted this revision.Aug 28 2017, 10:13 PM

This revision is now accepted and ready to land.Aug 28 2017, 10:13 PM

Harbormaster completed remote builds in B18194: Diff 44407.Aug 28 2017, 10:13 PM

Closed by commit rP0609133f4564: Limit notifications to latest 10, instead of 15 (authored by Chad Little <chad@phacility.com>). · Explain WhyAug 28 2017, 10:15 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

notification/

controller/

PhabricatorNotificationPanelController.php

2 lines

Diff 44407

View Options

src/applications/notification/controller/PhabricatorNotificationPanelController.php

Limit notifications to latest 10, instead of 15ClosedPublicActions