Closed
Bug 911980
Opened 11 years ago
Closed 11 years ago
[API] Refactor API code
Categories
(Participation Infrastructure :: Phonebook, defect)
Participation Infrastructure
Phonebook
Tracking
(Not tracked)
VERIFIED
FIXED
2013-09-12
People
(Reporter: giorgos, Assigned: giorgos)
References
Details
(Whiteboard: [kb=1100333] [qa-])
We need to refactor API code to not use the hacky ElasticSearch implementation and fix a bunch of bugs present in the current version. We won't be changing the interface (only adding fields to UserResource, as the list of collected user data grows) so we're not bumping the version.
Assignee | ||
Comment 1•11 years ago
|
||
And yes, add more and better tests!
Assignee | ||
Updated•11 years ago
|
OS: Linux → All
Hardware: x86_64 → All
Whiteboard: [kb=1100333]
Assignee | ||
Comment 2•11 years ago
|
||
Also explicitly set the fields returned by UserResource.
Comment 3•11 years ago
|
||
Commit pushed to master at https://github.com/mozilla/mozillians https://github.com/mozilla/mozillians/commit/a814dc7f4b5a7157d6514ee0ced0e2e0f5fb63d9 [fix bug 911980][fix bug 848256][fix bug 848257] API refactoring, tests and bug fixing. * Refactor API to not use the ElasticSearch backend (bug 911980) * Explicitly set the fields returned by UserResource * Fix filters containing spaces (bug 848257) * Fix filters with commas (bug 848256) * Fix default data ordering (bug 914384)
Updated•11 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
Target Milestone: --- → 2013-09-12
Comment 4•11 years ago
|
||
Is there any way to verify this fix, or should it be qa-?
Comment 5•11 years ago
|
||
Typically api testing has been [qa-] - it's not part of our testplan atm.
Comment 6•10 years ago
|
||
Bumping to verified as [qa-] -- if testing coverage from QA is wanted please ping me/us and we'll wrangle up some lovely automated and exploratory tests
Status: RESOLVED → VERIFIED
Whiteboard: [kb=1100333] → [kb=1100333] [qa-]
You need to log in
before you can comment on or make changes to this bug.
Description
•