Page MenuHomePhabricator
Differential D13171

Support PhabricatorSpacesInterface in ApplicationSearch UI
ClosedPublic
Actions

Authored by epriestley on Jun 5 2015, 5:10 PM.
Tags
None
Referenced Files
F15465221: D13171.id31833.diff
Wed, Apr 2, 4:16 PM
F15465220: D13171.id31832.diff
Wed, Apr 2, 4:16 PM
F15460958: D13171.id.diff
Tue, Apr 1, 3:45 AM
F15430799: D13171.id31832.diff
Mon, Mar 24, 9:01 AM
F15417029: D13171.id31844.diff
Thu, Mar 20, 2:53 PM
F15411156: D13171.id31832.diff
Wed, Mar 19, 8:52 AM
F15400303: D13171.diff
Mon, Mar 17, 10:53 AM
F15389223: D13171.id.diff
Sat, Mar 15, 5:00 AM
View All Files
Subscribers
epriestley

Details

Reviewers
btrahan
Maniphest Tasks
T8441: Integrate Spaces into ApplicationSearch
Commits
Restricted Diffusion Commit
rPb5dfd34e4a89: Support PhabricatorSpacesInterface in ApplicationSearch UI
Summary

Ref T8441. Does what it says, provided other conditions (like using the new SearchField stuff) are fulfilled.

Test Plan

Screen Shot 2015-06-05 at 10.09.29 AM.png (940×1 px, 153 KB)

Screen Shot 2015-06-05 at 10.09.30 AM.png (940×1 px, 158 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 31832.Jun 5 2015, 5:10 PM
epriestley retitled this revision from to Support PhabricatorSpacesInterface in ApplicationSearch UI.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
epriestley added a task: T8441: Integrate Spaces into ApplicationSearch.
Herald added a subscriber: epriestley. · View Herald TranscriptJun 5 2015, 5:10 PM
epriestley updated this revision to Diff 31833.Jun 5 2015, 5:11 PM
  • Fix a warning when no spaces exist yet.
btrahan accepted this revision.Jun 5 2015, 6:19 PM
btrahan edited edge metadata.
This revision is now accepted and ready to land.Jun 5 2015, 6:19 PM
Closed by commit rPb5dfd34e4a89: Support PhabricatorSpacesInterface in ApplicationSearch UI (authored by epriestley, committed by epriestley). · Explain WhyJun 5 2015, 6:21 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 31844

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/paste/query/PhabricatorPasteSearchEngine.php

Loading...
View Options

src/applications/search/engine/PhabricatorApplicationSearchEngine.php

Loading...
View Options

src/applications/search/field/PhabricatorSearchSpacesField.php

Loading...
View Options

src/applications/spaces/phid/PhabricatorSpacesNamespacePHIDType.php

Loading...
View Options

src/applications/spaces/query/PhabricatorSpacesNamespaceQuery.php

Loading...
View Options

src/applications/spaces/typeahead/PhabricatorSpacesNamespaceDatasource.php

Loading...
Phabricator · Built by Phacility