From 36dba82224de1b64bf571ebae5e944ce8a02e7c2 Mon Sep 17 00:00:00 2001 From: Valerio Bozzolan Date: Sat, 11 Feb 2023 11:10:58 +0100 Subject: [PATCH] Show the API summary on the top of each Conduit API page Summary: Show the API summary on the top of each Conduit API page. Before this change, the summary was only displayed from the Conduit APIs list. Closes T15141 Test Plan: I've opened 20 random Conduit API pages from the web interface and I checked that now there is an amazing "Summary" field. I also double-checked that there were not green peppers. Reviewers: O1 Blessed Committers, avivey Reviewed By: O1 Blessed Committers, avivey Subscribers: speck, tobiaswiese, Matthew, Cigaryno Maniphest Tasks: T15141 Differential Revision: https://we.phorge.it/D25063 --- .../controller/PhabricatorConduitConsoleController.php | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/applications/conduit/controller/PhabricatorConduitConsoleController.php b/src/applications/conduit/controller/PhabricatorConduitConsoleController.php index 70a2253084..b3dfa805a2 100644 --- a/src/applications/conduit/controller/PhabricatorConduitConsoleController.php +++ b/src/applications/conduit/controller/PhabricatorConduitConsoleController.php @@ -247,6 +247,10 @@ final class PhabricatorConduitConsoleController )); } + $view->addProperty( + pht('Summary'), + $method->getMethodSummary()); + $view->addProperty( pht('Returns'), $method->getReturnType());