Closed
Bug 1064968
Opened 10 years ago
Closed 8 years ago
Deprecate CurrentVersions, CurrentProducts and ProductsVersions api endpoints
Categories
(Socorro :: Webapp, task)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: peterbe, Assigned: peterbe)
References
Details
Attachments
(1 file)
CurrentVersions is just a wrapper on top of CurrentProducts that reshuffles the output slightly. ProductsVersions is just a wrapper on top of CurrentVersions that reshuffles the output slightly. The goal of the models was to be a thin "port" before going into the middleware. But maintaining them as individual endpoint-looking things comes at a maintenance cost. Now we need to worry about three versions of API_WHITELISTs. We should just keep the CurrentProducts model and maybe add some utility functions that does to CurrentProducts what CurrentVersions currently does.
Assignee | ||
Comment 1•10 years ago
|
||
You can see here, that CurrentVersions does with the output of CurrentProducts https://github.com/mozilla/socorro/pull/2343/files#diff-a7d1c53ffed84fe4a5fee1abc9bd61eaR250
Assignee: nobody → peterbe
Comment 2•9 years ago
|
||
See also: https://bugzilla.mozilla.org/show_bug.cgi?id=922335 We may want to close that out when this is fixed.
Assignee | ||
Updated•8 years ago
|
Depends on: 1246256
Summary: Deprecate CurrentVersions and ProductsVersions models → Deprecate CurrentVersions, CurrentProducts and ProductsVersions api endpoints
Assignee | ||
Comment 3•8 years ago
|
||
Comment 4•8 years ago
|
||
Commits pushed to master at https://github.com/mozilla/socorro https://github.com/mozilla/socorro/commit/49a70b5f202518b4e8d1fa2ceea235a34932cfb4 fixes bug 1064968 - Deprecate CurrentVersions, CurrentProducts and ProductsVersions https://github.com/mozilla/socorro/commit/56f65df0c7d2e8060f5d86cfac3918ca2efa4533 Merge pull request #3255 from peterbe/bug-1064968-deprecate-currentversions-currentproducts-and-productsversions-api-endpoints fixes bug 1064968 - Deprecate CurrentVersions, CurrentProducts and Pr…
Updated•8 years ago
|
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•