Closed Bug 911980 Opened 11 years ago Closed 11 years ago

[API] Refactor API code

Categories

(Participation Infrastructure :: Phonebook, defect)

defect
Not set
normal

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.
And yes, add more and better tests!
OS: Linux → All
Hardware: x86_64 → All
Whiteboard: [kb=1100333]
Blocks: 848257
Blocks: 848256
Blocks: 908702
Blocks: 862058
Also explicitly set the fields returned by UserResource.
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)
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2013-09-12
Is there any way to verify this fix, or should it be qa-?
Typically api testing has been [qa-] - it's not part of our testplan atm.
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.