Page MenuHomePhabricator

Audit - fix issue "showing older" on some commits

Authored by btrahan on Jan 23 2015, 7:31 PM.
Referenced Files
F11795517: D11479.diff
Sat, Jun 3, 3:09 PM
F11795257: D11479.id27611.diff
Sat, Jun 3, 11:19 AM
Unknown Object (File)
Tue, May 30, 3:02 PM
Unknown Object (File)
May 3 2023, 5:46 PM
Unknown Object (File)
May 3 2023, 5:46 PM
Unknown Object (File)
Apr 14 2023, 7:55 PM
Unknown Object (File)
Apr 9 2023, 5:00 AM
Unknown Object (File)
Apr 7 2023, 11:25 PM



Fixes T7021. When I moved around all the timeline stuff I guess I didn't find this "corner" case, which is wildly common in the post-commit review workflow that we don't use.

Test Plan

pre-patch I could reproduce the issue and post patch I could not. The reproduction case is to have a commit with inline comments and then enough subsequent comments to have a "show older" UI. clicking "show older" now works!

Diff Detail

rP Phabricator
Lint Passed
Tests Passed
Build Status
Buildable 4018
Build 4031: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

btrahan retitled this revision from to Audit - fix issue "showing older" on some commits.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Jan 23 2015, 7:31 PM
This revision was automatically updated to reflect the committed changes.