Page MenuHomePhabricator

Provide bucketing for commits in Audit
ClosedPublic

Authored by epriestley on Jan 12 2017, 6:31 PM.
Tags
None
Referenced Files
F18817700: D17192.id41342.diff
Tue, Oct 21, 5:39 PM
F18783278: D17192.id.diff
Mon, Oct 13, 5:42 AM
F18780780: D17192.diff
Sun, Oct 12, 7:51 PM
F18684144: D17192.id.diff
Fri, Sep 26, 8:45 AM
F18613891: D17192.id41342.diff
Sep 14 2025, 11:35 AM
F18600220: D17192.id.diff
Sep 13 2025, 9:15 AM
F18595189: D17192.id41342.diff
Sep 12 2025, 8:48 PM
F18591440: D17192.id.diff
Sep 12 2025, 9:14 AM
Subscribers
None

Details

Summary

Fixes T9430. Fixes T9362. Fixes T9544. This changes the default view of Audit to work like Differential, where commits you need to audit or respond to are shown in buckets.

This is a bit messy and probably needs some followups. This stuff has changed from a compatibility viewpoint:

  • The query works differently now (but in a better, modern way), so existing saved queries will need to be updated.
  • I've removed the counters from the home page instead of updating them, since they're going to get wiped out by ProfileMenu soon anyway.
  • When bucketed queries return too many results (more than 1,000) we now show a warning about it. This isn't greaaaat but it seems good enough for now.
Test Plan

Screen Shot 2017-01-12 at 10.16.55 AM.png (1×1 px, 365 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable