Page MenuHomePhabricator

Use new Differential bucketing logic on default (non-dashboard) homepage
ClosedPublic

Authored by epriestley on May 19 2016, 8:32 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Apr 7, 11:12 AM
Unknown Object (File)
Thu, Apr 4, 4:33 PM
Unknown Object (File)
Sat, Mar 30, 1:58 AM
Unknown Object (File)
Wed, Mar 20, 10:56 AM
Unknown Object (File)
Feb 20 2024, 8:00 AM
Unknown Object (File)
Feb 9 2024, 2:20 AM
Unknown Object (File)
Feb 3 2024, 10:11 AM
Unknown Object (File)
Jan 23 2024, 7:53 AM
Subscribers
None

Details

Summary

Ref T10939. If you haven't installed a dashboard, we show an "Active Revisions" panel on the homepage by default. I waited a bit to update this, but the new buckets don't seem to have caused any major problems so far.

Update this to use the new logic. I'm just showing "must review" + "should review", which is similar to the old beahvior.

Also replace the notification count with this same number. This is a little different from the old behavior, but simpler, and I think we should probably move toward getting rid of these counts completely.

Test Plan
  • Viewed homepage as logged-in user, saw my revisions (including revisions I have authority over only because of project membership).
  • Saw consistent notification count.
  • Grepped for removed method.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Use new Differential bucketing logic on default (non-dashboard) homepage.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.May 19 2016, 10:07 PM
This revision was automatically updated to reflect the committed changes.