Page MenuHomePhabricator
Differential D11302

Projects - tokenize [ProjectX] so "projectX" is a match
ClosedPublic
Actions

Authored by btrahan on Jan 9 2015, 10:07 PM.
Tags
None
Referenced Files
F18746841: D11302.id27156.diff
Fri, Oct 3, 1:38 PM
F18712363: D11302.id.diff
Mon, Sep 29, 5:22 AM
F18623141: D11302.id.diff
Mon, Sep 15, 2:29 PM
F18622652: D11302.diff
Mon, Sep 15, 12:35 PM
F18621171: D11302.id27157.diff
Mon, Sep 15, 7:16 AM
F18616810: D11302.id27157.diff
Sun, Sep 14, 7:07 PM
F18616769: D11302.id27157.diff
Sun, Sep 14, 6:55 PM
F18616604: D11302.id27156.diff
Sun, Sep 14, 6:17 PM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Maniphest Tasks
T4656: Project name fuzzy search does not work with brackets
Commits
Restricted Diffusion Commit
rP59770443b62f: Projects - tokenize [ProjectX] so "projectX" is a match
Summary

Fixes T4656. Helps users with this naming convention, which is probably not super duper rare.

Users will need to make an edit to a project -or- run bin/search index "#project-tag" to make this actually work.

Test Plan

made a project "[T4656test]". Typed "t4" and project showed up!

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

btrahan updated this revision to Diff 27156.Jan 9 2015, 10:07 PM
btrahan added a task: T4656: Project name fuzzy search does not work with brackets.
btrahan retitled this revision from to Projects - tokenize [ProjectX] so "projectX" is a match.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
Herald added subscribers: epriestley, Korvin. · View Herald TranscriptJan 9 2015, 10:07 PM
btrahan updated this object.Jan 9 2015, 10:08 PM
epriestley accepted this revision.Jan 9 2015, 10:08 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Jan 9 2015, 10:08 PM
Closed by commit rP59770443b62f: Projects - tokenize [ProjectX] so "projectX" is a match (authored by btrahan, committed by btrahan). · Explain WhyJan 9 2015, 10:09 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
resources/
celerity/
12 lines
src/
applications/
typeahead/
datasource/
2 lines
webroot/
rsrc/
externals/
javelin/
lib/
control/
typeahead/
normalizer/
2 lines

Diff 27157

View Options

resources/celerity/map.php

Loading...
View Options

src/applications/typeahead/datasource/PhabricatorTypeaheadDatasource.php

Loading...
View Options

webroot/rsrc/externals/javelin/lib/control/typeahead/normalizer/TypeaheadNormalizer.js

Loading...
Phabricator · Built by Phacility