Skip to content

feat(match_phrase): Use match_phrase lib#111

Merged
orangejulius merged 1 commit intomasterfrom
use-match-leaf-function
Oct 3, 2019
Merged

feat(match_phrase): Use match_phrase lib#111
orangejulius merged 1 commit intomasterfrom
use-match-leaf-function

Conversation

@orangejulius
Copy link
Copy Markdown
Member

@orangejulius orangejulius commented Oct 3, 2019

This uses the match_phrase helper function defined in #109 to cut down on duplication when creating match_phrase queries.

Since the structure of the match_phrase query is defined in one place, it makes the resulting view code a bit more concise.

It will also make optional parameter handling easier and more consistent.

This uses the match_phrase helper function defined in
#109 to cut down on duplication when
creating `match_phrase` queries.

Since the structure of the `match_phrase` query is defined in one place,
it makes the resulting view code a bit more concise.

It will also make optional parameter handling easier and more
consistent.
@orangejulius orangejulius merged commit 4d0298f into master Oct 3, 2019
@orangejulius orangejulius deleted the use-match-leaf-function branch October 3, 2019 17:28
orangejulius added a commit that referenced this pull request Oct 3, 2019
This change builds on the previous series of PRs
(#112,
and #111) to support the `slop`
parameter on the `match_phrase` queries responsible for handling street
addresses.

Replaces #101
Connects pelias/api#1322
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant