Closed
Bug 857055
Opened 12 years ago
Closed 12 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•12 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•12 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•12 years ago
|
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
| Assignee | ||
Updated•12 years ago
|
Target Milestone: --- → 42
Comment 3•12 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
•