Page MenuHomePhabricator

Merge "Audit" more completely into "Diffusion"
ClosedPublic

Authored by epriestley on Jan 11 2017, 11:20 PM.
Tags
None
Referenced Files
F19637491: D17186.diff
Thu, Feb 5, 2:25 PM
F19619469: D17186.id41331.diff
Wed, Feb 4, 9:48 AM
F19605610: D17186.diff
Tue, Feb 3, 9:34 AM
F19526161: D17186.diff
Sun, Jan 18, 2:28 PM
F18872373: D17186.diff
Nov 5 2025, 3:53 AM
F18839157: D17186.id.diff
Oct 27 2025, 2:44 PM
F18839103: D17186.diff
Oct 27 2025, 2:27 PM
F18763375: D17186.id.diff
Oct 6 2025, 11:58 PM
Subscribers
None

Details

Summary

Fixes T6630. Long ago, "Audit", "Diffusion" and "Repositories" were three totally separate applications.

This separation isn't useful and the three rapidly became intertwined. Ideally, they would all be one application.

This doesn't take us quite that far, but Audit no longer has any controllers and has little actual behavior.

The "Audit" screen has always just been a SearchEngine view of commits with some filters on it, and this formalizes that and puts a link to it in Diffusion. (This view has other uses, too.)

Test Plan
  • Accessed audit from home page.
  • Accessed audit/commits from Diffusion.
  • Could no longer uninstall Audit on its own.
  • Grepped for /audit/ and AuditApplication.

Diff Detail

Repository
rP Phabricator
Branch
commit11
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 15216
Build 19999: Run Core Tests
Build 19998: arc lint + arc unit

Event Timeline

This revision is now accepted and ready to land.Jan 11 2017, 11:43 PM
This revision was automatically updated to reflect the committed changes.