Leaderboard json #147

Merged
merged 4 commits into from Aug 10, 2013

Conversation

Projects
None yet
3 participants
Owner

relrod commented Aug 10, 2013

This pulls out the code that generated JSON for /user/<id>/json into something that other methods can call, then adds a JSON api that includes that info along with an ordered array of users which forms the leaderboard.

@@ -133,6 +133,8 @@ def get_db(request):
config.add_route('builder', '/builder')
config.add_route('explore', '/explore')
config.add_route('leaderboard', '/leaderboard')
+ config.add_route('leaderboard_json', '/leaderboard/json')
+ #config.add_route('rank_json', '/leaderboard/{id}/json')
@puiterwijk

puiterwijk Aug 10, 2013

Owner

Why is this commented out?

@relrod

relrod Aug 10, 2013

Owner

(answered in IRC - this endpoint will exist soon, I just wanted to note it here since it's related to this change, so I added it at the same time, but commented it.)

Contributor

thedavidgay commented Aug 10, 2013

This is awesome. :) Thanks!

thedavidgay added a commit that referenced this pull request Aug 10, 2013

@thedavidgay thedavidgay merged commit 31e9d61 into develop Aug 10, 2013

1 check failed

default PEP8bot detected 191 errors
Details

@thedavidgay thedavidgay deleted the leaderboard_json branch Aug 10, 2013

Owner

puiterwijk commented Aug 10, 2013

oh, I'm too late.
Anyway: 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment