Status

Participation Infrastructure
Phonebook
RESOLVED FIXED
4 years ago
3 years ago

People

(Reporter: hoosteeno, Unassigned)

Tracking

(Blocks: 2 bugs)

Details

(Whiteboard: [kb=1258286] )

As bug 845593 comment 4 mentions, we need to change our API library in order to minimize the impact of keeping Django up to date. In this bug let's ...

* identify the library and version we wish to use
* make a branch for that work and work on it
* discuss how to integrate/launch
* build any blocking bugs (docs changes? api versioning? cleaning up endpoints?) required
Whiteboard: [kb=1258286]
My plan is to:
 1. Upgrade tastypie to a django 1.5 compatible version
 2. Upgrade to django >= 1.5 (bug 845593)
 3. Move on with this bug, evaluating different libraries and the overall API future.

--

For (1) as mentioned in bug 845593 comment 4 a few changes are needed. I've resolved some of them but some seem more complex. All the unresolved issues are linked to the 'city' and 'country' resources.

Those two endpoints, along with Group, Skill and Language endpoints were added before the 2013 Summit to support the summit app. These are undocumented and because of that I highly doubt if anyone is using them atm. 

I suggest that we drop those endpoints and thus unblock (1) and (2). Then we plan the API future and we decide if we properly support those endpoints or not.

What do you think?
Flags: needinfo?(hoosteeno)
Giorgos, let's verify the usage of those endpoints by monitoring them for a week (using graphite?) and then move forward with your plan above.
Flags: needinfo?(hoosteeno)
(In reply to Giorgos Logiotatidis [:giorgos] from comment #1)
> Those two endpoints, along with Group, Skill and Language endpoints were
> added before the 2013 Summit to support the summit app. These are
> undocumented and because of that I highly doubt if anyone is using them atm. 

I was wrong about those being undocumented. There is documentation in the official docs. In any case we'll monitor them before acting.
Depends on: 978694
In meeting, Giorgos reported that...

UserResource being used daily
GroupResource was used once (possibly by the FirefoxOS Tablet program)
City, Country, Skill were never used

Let's get rid of the endpoints that were never used and plan to re-implement them after the bigger API authorization refactor is done (or as part of it).
Blocks: 989352
Status: NEW → ASSIGNED

Comment 5

4 years ago
Commit pushed to master at https://github.com/mozilla/mozillians

https://github.com/mozilla/mozillians/commit/a00ea48308ca624a98e7c71bb46e7362eeeaa3fe
[bug 965828] Tastypie upgrade.

- Upgrade tastypie to 0.9.16
- Remove Country and City resources.
Blocks: 1011209
De-assigning myself since I'm not working on mozillians now.
Assignee: giorgos → nobody
Status: ASSIGNED → NEW
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.