Page MenuHomePhabricator
Differential D16446

Added application name to the typeahead results for doc type search
ClosedPublic
Actions

Authored by jcox on Aug 24 2016, 10:11 PM.
Tags
None
Referenced Files
F13162593: D16446.diff
Mon, May 6, 2:28 PM
Unknown Object (File)
Sun, May 5, 5:01 AM
Unknown Object (File)
Fri, May 3, 3:31 AM
Unknown Object (File)
Mon, Apr 29, 3:20 PM
Unknown Object (File)
Mon, Apr 29, 8:14 AM
Unknown Object (File)
Sat, Apr 27, 12:56 AM
Unknown Object (File)
Wed, Apr 24, 11:11 PM
Unknown Object (File)
Sat, Apr 20, 10:41 PM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T10951: Search shows document types corresponding to disabled apps
Commits
rPd135b3f2d56f: Added application name to the typeahead results for doc type search
Summary

Ref T10951. This adds the application name as an attribute below the document type in the UI for doc type search.

Test Plan

Verify that the application name appears as an attribute on the document type results.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

jcox updated this revision to Diff 39551.Aug 24 2016, 10:11 PM
jcox retitled this revision from to Added application name to the typeahead results for doc type search.
jcox updated this object.
jcox edited the test plan for this revision. (Show Details)
jcox added a task: T10951: Search shows document types corresponding to disabled apps.
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptAug 24 2016, 10:11 PM
Herald added a subscriber: epriestley. · View Herald Transcript
epriestley accepted this revision.Aug 25 2016, 2:21 PM
epriestley added a reviewer: epriestley.
This revision is now accepted and ready to land.Aug 25 2016, 2:21 PM
jcox updated this revision to Diff 39561.Aug 25 2016, 3:16 PM
jcox edited edge metadata.

Made line shorter

Herald added a subscriber: Korvin. · View Herald TranscriptAug 25 2016, 3:16 PM
Closed by commit rPd135b3f2d56f: Added application name to the typeahead results for doc type search (authored by jcox, committed by jcox). · Explain WhyAug 25 2016, 3:26 PM
This revision was automatically updated to reflect the committed changes.
epriestley mentioned this in D16459: Converted the pinned applications selector to a typeahead..Aug 26 2016, 5:22 PM

Revision Contents
Changeset List

PathSize
src/
applications/
search/
typeahead/
16 lines

Diff 39562

View Options

src/applications/search/typeahead/PhabricatorSearchDocumentTypeDatasource.php

Loading...
Phabricator · Built by Phacility