Skip to content
This repository has been archived by the owner on Jan 30, 2019. It is now read-only.

Fix profile index #53

Merged
merged 3 commits into from
Jul 17, 2014
Merged

Fix profile index #53

merged 3 commits into from
Jul 17, 2014

Conversation

ckrailo
Copy link
Member

@ckrailo ckrailo commented Jul 16, 2014

Fixes issue #51. See there for details. :)

ckrailo added 2 commits July 15, 2014 21:02
@ckrailo
Remove elipsis from pagination bar.
8262f94 
Part of issue #51.
@ckrailo
Change epic_sort algorithm.
4740587 
This method should penalize people a ton if they don't have names and a
little for each other profile piece they're missing.

Part of issue #51.
@ckrailo
Copy link
Member Author

ckrailo commented Jul 16, 2014

Pinging @itchy for review since you wrote the original epic_sort.

itchy added a commit that referenced this pull request Jul 17, 2014
@itchy
Merge pull request #53 from dallasrb/fix_profile_index
5cd643a 
Fix profile index - update tests to match new sort algorithm
@itchy itchy merged commit 5cd643a into master Jul 17, 2014
@itchy
Copy link
Contributor

itchy commented Jul 17, 2014

Thanks Krailo -- changes look great -- I just updated the tests and merged
the branch.

On Tue, Jul 15, 2014 at 9:34 PM, Christopher Krailo <
[email protected]> wrote:

Pinging @itchy https://github.com/itchy for review since you wrote the
original epic_sort.

—
Reply to this email directly or view it on GitHub
#53 (comment).

Scott Johnson

@itchy itchy deleted the fix_profile_index branch July 17, 2014 14:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ckrailo @itchy