Closed
Bug 857055
Opened 11 years ago
Closed 11 years ago
[PHP App] Fix forcing implementation of a middleware service
Categories
(Socorro :: Webapp, task)
Socorro
Webapp
Tracking
(Not tracked)
VERIFIED
FIXED
42
People
(Reporter: adrian, Assigned: adrian)
References
Details
We changed the way implementation forcing worked in bug 846347, but didn't change the current UI to reflect those changes. If we want to efficiently test search with elasticsearch, we need to fix the old PHP app.
Assignee | ||
Comment 1•11 years ago
|
||
Pull request: https://github.com/mozilla/socorro/pull/1173 Steps to QA ----------- 1. Go to advanced search 2. Run a search returning more than 1,000 results 3. Add at the end of the URL "&_force_api_impl=elasticsearch" 4. Verify that you get exactly 1,000 results
Comment 2•11 years ago
|
||
Commits pushed to master at https://github.com/mozilla/socorro https://github.com/mozilla/socorro/commit/88f223aeebe70f01970aa581f6f7fc2048b45503 Fixes bug 857055 - Fixed implementation forcing in PHP Web App. https://github.com/mozilla/socorro/commit/0429434601853a2c10239ff24eb148dfb3ef8aad Merge pull request #1173 from AdrianGaudebert/857055-force-impl-old-webapp Fixes bug 857055 - Fixed implementation forcing in PHP Web App.
Updated•11 years ago
|
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•11 years ago
|
Target Milestone: --- → 42
Comment 3•11 years ago
|
||
QA verified on stage - awesome steps to test in comment 1. Thanks Adrian. Returns 26,961 results: https://crash-stats.allizom.org/query/query?product=Firefox&version=ALL%3AALL&range_value=1&range_unit=weeks&date=04%2F17%2F2013+17%3A07%3A48&query_search=signature&query_type=contains&query=&reason=&build_id=&process_type=any&hang_type=any&do_query=1 Returns 1,000 results: https://crash-stats.allizom.org/query?product=Firefox&query_search=signature&query_type=contains&reason_type=contains&date=04%2F17%2F2013%2017%3A07%3A48&range_value=1&range_unit=weeks&hang_type=any&process_type=any&do_query=1&_force_api_impl=elasticsearch
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•