Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Enable support for multiple source fields in the tokenizer #811

Closed
wants to merge 1 commit into from
Closed

Enable support for multiple source fields in the tokenizer #811

wants to merge 1 commit into from

Conversation

dr3x
Copy link
Contributor

@dr3x dr3x commented Apr 2, 2014

Allow multiple source fields, separated by space:

Before:
datumTokenizer: Bloodhound.tokenizers.obj.whitespace('name'),

After:
datumTokenizer: Bloodhound.tokenizers.obj.whitespace('name description'),

@dr3x dr3x changed the title Enable support for multiple fields Enable support for multiple source fields in the tokenizer Apr 2, 2014
@jharding jharding added this to the v0.10.3 milestone Apr 5, 2014
@jharding
Copy link
Contributor

jharding commented Apr 5, 2014

Ah, good thinking. Since property names can have spaces in them though, how about we support something like this:

Bloodhound.tokenizers.obj.whitespace('name', 'description');

@jharding jharding modified the milestones: v0.10.3, v0.10.4 Jul 9, 2014
@jharding
Copy link
Contributor

jharding commented Jul 9, 2014

Merged into integration-0.10.3 as bca7441.

@jharding jharding closed this Jul 9, 2014
@felixbuenemann felixbuenemann mentioned this pull request Jul 13, 2014
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
bloodhound
Milestone
v0.10.3
Development

Successfully merging this pull request may close these issues.
2 participants
@dr3x @jharding