Closed
Bug 1025160
Opened 10 years ago
Closed 10 years ago
[dev][stage][regression] location based search results are incorrect
Categories
(Participation Infrastructure :: Phonebook, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: mbrandt, Assigned: sancus)
References
Details
(Whiteboard: [fromAutomation])
Location based result sets appear to be incorrect and overly broad. Result sets include profiles that aren't specific to that /region/ or /city/ but share the same /country/ [afaict] Steps to reproduce goto https://mozillians.allizom.org/en-US/country/United%20States/region/Colorado/ goto https://mozillians.allizom.org/en-US/country/United%20States/city/Waterflow/ Expected: Results only include accounts that match the geographic search criteria Actual: Search results include profiles that match /country/ but have other field blank
Reporter | ||
Updated•10 years ago
|
Whiteboard: [fromAutomation]
Reporter | ||
Comment 2•10 years ago
|
||
xfailing the failing testcase https://github.com/mozilla/mozillians-tests/pull/154/files
Reporter | ||
Updated•10 years ago
|
Summary: [stage][regression] location based search results are incorrect → [dev][stage][regression] location based search results are incorrect
Reporter | ||
Updated•10 years ago
|
Severity: major → blocker
Assignee | ||
Comment 3•10 years ago
|
||
https://github.com/mozilla/mozillians/pull/975 is up for this.
Comment 4•10 years ago
|
||
Commits pushed to master at https://github.com/mozilla/mozillians https://github.com/mozilla/mozillians/commit/6369c2b1eb35bf913a841839fccc30743d3598c6 [Fix Bug 1025160] Location list pages should only display profiles that match all arguments. https://github.com/mozilla/mozillians/commit/8593d101f735c93c34e1ac61e37a2afbfdd6fef6 Merge pull request #975 from Sancus/1026258-location-list [Fix Bug 1025160] Location list pages should only display profiles that match all arguments.
Updated•10 years ago
|
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Updated•10 years ago
|
Version: other → next
Reporter | ||
Comment 5•10 years ago
|
||
Bumping to QA verified - automation passes on dev .. removing xfail in prep for this to be staged.
Status: RESOLVED → VERIFIED
Updated•10 years ago
|
Assignee: nobody → sancus
Updated•10 years ago
|
Version: next → 2014-06.1
You need to log in
before you can comment on or make changes to this bug.
Description
•