Differential D11963

Show "Show Older" for transactions the user has not interacted with
ClosedPublic
Actions

Authored by epriestley on Mar 4 2015, 12:19 PM.

Tags

None

Referenced Files

	F14036071: D11963.diff
	Sun, Nov 10, 8:26 AM

	F13995723: D11963.id28812.diff
	Wed, Oct 23, 2:59 PM

	Unknown Object (File)
	Sep 23 2024, 2:34 PM

	Unknown Object (File)
	Sep 21 2024, 12:53 AM

	Unknown Object (File)
	Sep 16 2024, 9:28 AM

	Unknown Object (File)
	Sep 9 2024, 6:43 PM

	Unknown Object (File)
	Sep 5 2024, 2:57 AM

	Unknown Object (File)
	Aug 28 2024, 10:11 AM

Subscribers

Details

Reviewers

Maniphest Tasks

T7454: Longer Maniphest tasks not fully displayed (as Phabricator defines a result set limit as 100 rows)

Commits

Restricted Diffusion Commit
rPb09168e689ff: Show "Show Older" for transactions the user has not interacted with

Summary

Fixes T7454. We weren't triggering "Show Older" unless there were hidden events because of a previous interaction.

Test Plan

Set page size to 3.
Viewed a task with ~10 transactions.
Before patch:
- Only 3 most recent transactions visible, no way to see older ones.
- Saw "show older" appear, paged backward through transaction histroy.
Also, interacted with task and then viewed it, made sure "show older" still works.

Diff Detail

Repository

Branch

older

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 4741
Build 4755: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley updated this revision to Diff 28804.Mar 4 2015, 12:19 PM

epriestley retitled this revision from to Show "Show Older" for transactions the user has not interacted with.

epriestley updated this object.

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

epriestley added a reviewer: btrahan.

epriestley added a task: T7454: Longer Maniphest tasks not fully displayed (as Phabricator defines a result set limit as 100 rows).

Herald added a subscriber: epriestley. · View Herald TranscriptMar 4 2015, 12:19 PM

btrahan accepted this revision.Mar 4 2015, 6:25 PM

btrahan edited edge metadata.

This revision is now accepted and ready to land.Mar 4 2015, 6:25 PM

Closed by commit rPb09168e689ff: Show "Show Older" for transactions the user has not interacted with (authored by epriestley, committed by epriestley). · Explain WhyMar 4 2015, 6:36 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

view/

control/

AphrontCursorPagerView.php

4 lines

phui/

PHUITimelineView.php

21 lines

Diff 28804

src/view/control/AphrontCursorPagerView.php

Loading...

src/view/phui/PHUITimelineView.php

Loading...