mendicant-original / puzzlenode

Quiz application inspired by Project Euler and the Internet Problem Solving Contest (IPSC)
puzzlenode.com
81 stars 43 forks source link

Add local neighborhood to leaderboard #39

Closed madebydna closed 12 years ago

madebydna commented 13 years ago

Show the 5 people above and the 4 people below the current_user

mikbe commented 12 years ago

I should be able to do this one. Just to be clear you mean something like this:

Leaderboard Neighborhood

And if they are ranked say 15th the list would fold into the top ten list continuing to the 19th ranked.

abacha commented 12 years ago

I've started working on this as my CS on RMU

jordanbyron commented 12 years ago

Hey @mikbe that looks about right, but as @abacha pointed out he is currently working on this for his January core course. Thanks for doing the mock though!

abacha commented 12 years ago

Hey @jordanbyron, I've started working on this issue as you can see here: https://github.com/abacha/puzzlenode/commit/e41cd93600df17e832a3189353ed754ce3eb60d4 I thought about defining the range of neighboors (fixed 9 in the code) on a constant or maybe a default parameter to the method, I'll refactor that later but I'd like some opinion on my approach to the problem

jordanbyron commented 12 years ago

@abacha please send over a pull request so we can discuss your code there.

jordanbyron commented 12 years ago

Closed with pull request #68