763624 - Make location searchable

Status: VERIFIED FIXED

(Participation Infrastructure :: Phonebook, defect)

Description

[James Socol [:jsocol, :james]]

Once we have location, we'd really like to make it possible to search by location. There are some mockups on bug 657071 but I think we may want to make this a little more iterative, so for this bug let's just make sure the site search includes it.

Comment 1

[Aakash Desai [:aakashd]]

Here is the feature wiki for location-based searches:

https://wiki.mozilla.org/Mozillians/Phonebook/Search#.05_-_Location_Results

Comment 2

[Aakash Desai [:aakashd]]

Ok, I've done some investigation and here's what I can tell:

* We need to update Mozillian's ElasticSearch instance to the latest release of ElasticUtils (bug 783082)
* Use term-phrase query on country, region and city

Comment 3

[[github robot]]

Commits pushed to master at https://github.com/mozilla/mozillians

https://github.com/mozilla/mozillians/commit/491f18d549e8797de69ac973092e0b786bacb482
[fix bug 763624] Make location (country, region, city) searchable.

https://github.com/mozilla/mozillians/commit/598ec9efb894b19aea76f55a0dbae84be002163b
Merge pull request #319 from glogiotatidis/763624

[fix bug 763624] Make location (country, region, city) searchable.

Comment 4

[Luke Crouch [:groovecoder]]

omg awesome. when will this land on the live site?

Comment 5

[Aakash Desai [:aakashd]]

Likely Wednesday/Thursday of next week.

Comment 6

[Giorgos Logiotatidis [:giorgos]]

We need to use text_phrase instead of query_string for location search. This is fixed on the latest elasticutils, commit b318e9.

Comment 7

[[github robot]]

Commit pushed to master at https://github.com/mozilla/mozillians

https://github.com/mozilla/mozillians/commit/82d4c550bd60ad57fcb5cd8b7a155b6d9f3c6e54
[fix bug 763624] Use text_phrase instead of query_string for location search.

Comment 9

[Ben (:bensternthal)]

testing on dev looks good

Comment 10

[Rebecca Billings [:rbillings]]

Verified able to search on location