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
Unknown Object (File)
Tue, Jan 28, 5:35 AM
Unknown Object (File)
Fri, Jan 24, 3:41 PM
Unknown Object (File)
Fri, Jan 24, 3:41 PM
Unknown Object (File)
Fri, Jan 24, 3:41 PM
Unknown Object (File)
Fri, Jan 24, 3:40 PM
Unknown Object (File)
Wed, Jan 22, 10:48 AM
Unknown Object (File)
Tue, Jan 21, 9:54 AM
Unknown Object (File)
Dec 30 2024, 5:01 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
Branch
AddAppNameToDocTypeResults (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 13427
Build 17249: Run Core Tests
Build 17248: arc lint + arc unit

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 39551

View Options

src/applications/search/typeahead/PhabricatorSearchDocumentTypeDatasource.php

Loading...
Phabricator · Built by Phacility