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
F15472174: D11302.diff
Sat, Apr 5, 11:14 AM
F15435825: D11302.id27156.diff
Tue, Mar 25, 10:27 AM
F15400679: D11302.id27157.diff
Mon, Mar 17, 1:18 PM
F15389588: D11302.id.diff
Mar 15 2025, 5:29 AM
F15352966: D11302.id27157.diff
Mar 10 2025, 7:17 PM
F15338841: D11302.diff
Mar 9 2025, 1:05 PM
Unknown Object (File)
Feb 8 2025, 12:59 AM
Unknown Object (File)
Feb 8 2025, 12:59 AM
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