Closed
Bug 713871
Opened 13 years ago
Closed 13 years ago
Move TCBS service to new-style Middleware
Categories
(Socorro Graveyard :: Middleware, defect)
Socorro Graveyard
Middleware
Tracking
(Not tracked)
VERIFIED
FIXED
2.4.1
People
(Reporter: adrian, Assigned: adrian)
References
Details
The existing TCBS service is not easily extensible and not really clean. As we want to add a new argument per bug 641487, I think it is time to rewrite that TCBS service.
Updated•13 years ago
|
Component: Socorro → General
Product: Webtools → Socorro
Assignee | ||
Comment 1•13 years ago
|
||
Opened pull request: https://github.com/mozilla/socorro/pull/248
Assignee | ||
Updated•13 years ago
|
Component: General → Middleware
QA Contact: socorro → middleware
Assignee | ||
Updated•13 years ago
|
Target Milestone: 2.4 → 2.4.1
Comment 2•13 years ago
|
||
Commits pushed to https://github.com/mozilla/socorro
https://github.com/mozilla/socorro/commit/7c65baa01e550f3865bc9c4746bddaebb587e74b
Fixes bug 713871 - Made TCBS service a new-style middleware service. Moved SQL queries into the external module.
https://github.com/mozilla/socorro/commit/28dc663a837ee0656a26443feab1c6d9ca23154f
Merge pull request #248 from AdrianGaudebert/713871-tcbs-new-style-service
Fixes bug 713871 - Made TCBS service a new-style middleware service.
Updated•13 years ago
|
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 3•13 years ago
|
||
Steps to QA
-----------
1. Open any TCBS page on dev or stage.
2. Compare results with prod.
3. For similar dates, products, versions, results should be identical.
This bug did not change anything in the algorithm, only changed the organization of the code. The behavior of TCBS should not have changed from before.
Comment 4•13 years ago
|
||
QA verified. Thank you adrian for the helpful steps to test. This really helps the community understand which areas changed in the app.
Behavior between production and stage appear similar.
Status: RESOLVED → VERIFIED
Updated•9 years ago
|
Product: Socorro → Socorro Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•