Page MenuHomePhabricator
Differential D14779

Move ConduitLogs to ApplicationSearch
ClosedPublic
Actions

Authored by epriestley on Dec 14 2015, 8:34 PM.
Tags
None
Referenced Files
F15476711: D14779.id35747.diff
Mon, Apr 7, 8:54 AM
F15457706: D14779.diff
Sun, Mar 30, 6:00 PM
F15431141: D14779.diff
Mon, Mar 24, 10:46 AM
F15399535: D14779.id.diff
Mon, Mar 17, 5:04 AM
F15391733: D14779.diff
Sat, Mar 15, 11:23 AM
F15387348: D14779.id35740.diff
Sat, Mar 15, 2:15 AM
F15334932: D14779.diff
Mar 8 2025, 1:14 PM
Unknown Object (File)
Feb 27 2025, 11:01 AM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T9755: Have conduit call logs actually log the caller
T9980: Give administrators better tools for managing Conduit API changes
Commits
Restricted Diffusion Commit
rP4a147dcbfb15: Move ConduitLogs to ApplicationSearch
Summary

Ref T9980. Start making this UI more useful and powerful so we can give administrators a better toolset for reacting to API changes.

Fixes T9755. We were logging the caller, just not rendering it properly.

Test Plan

Screen Shot 2015-12-14 at 12.29.13 PM.png (1×1 px, 280 KB)

Diff Detail

Repository
rP Phabricator
Branch
clog2
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 9558
Build 11421: Run Core Tests
Build 11420: arc lint + arc unit

Event Timeline

epriestley updated this revision to Diff 35740.Dec 14 2015, 8:34 PM
epriestley retitled this revision from to Move ConduitLogs to ApplicationSearch.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added tasks: T9980: Give administrators better tools for managing Conduit API changes, T9755: Have conduit call logs actually log the caller.
chad accepted this revision.Dec 14 2015, 8:46 PM
chad edited edge metadata.
This revision is now accepted and ready to land.Dec 14 2015, 8:46 PM
Closed by commit rP4a147dcbfb15: Move ConduitLogs to ApplicationSearch (authored by epriestley, committed by epriestley). · Explain WhyDec 14 2015, 10:45 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 35740

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/conduit/application/PhabricatorConduitApplication.php

Loading...
View Options

src/applications/conduit/controller/PhabricatorConduitAPIController.php

Loading...
View Options

src/applications/conduit/controller/PhabricatorConduitLogController.php

Loading...
View Options

src/applications/conduit/query/PhabricatorConduitLogQuery.php

Loading...
View Options

src/applications/conduit/query/PhabricatorConduitLogSearchEngine.php

Loading...
Phabricator · Built by Phacility