Skip to content
This repository has been archived by the owner on Nov 13, 2021. It is now read-only.

Rails 4 branch not sorting index page by likes. #68

Open
jessewaites opened this issue Oct 17, 2013 · 7 comments
Open

Rails 4 branch not sorting index page by likes. #68

jessewaites opened this issue Oct 17, 2013 · 7 comments

Comments

@jessewaites
Copy link

Im using the Rails 4 branch of this and having issues - I think something is broken. Users can vote, and anyone can see the votes, but the index page does not sort at all. Controller- I'm using Can Can btw, in case that is relevant.

def index
@fonts = Font.find_with_reputation(:likes, :all, {:order => 'likes DESC'})
end

@eelcoj
Copy link

eelcoj commented Nov 20, 2013

Is there a solution for this? Seem to stumbled upon the same issue.

Duplicate of #62

Failing test of https://travis-ci.org/twitter/activerecord-reputation-system/jobs/6014002#L232 could be related.

@jessewaites
Copy link
Author

I never got it fixed. I have a feeling that it might be related to Can Can but I'm not sure.

@eelcoj
Copy link

eelcoj commented Nov 21, 2013

@piratebroadcast nope, don't use CanCan here.

@caiosba
Copy link
Contributor

caiosba commented Sep 8, 2014

FYI, I have a fork with a Rails 4 branch... all tests are passing and I'm using it on a Rails4 + Ruby 2 application without problems... you can try that and let me know if things are not working there: https://github.com/caiosba/activerecord-reputation-system/tree/rails4

@jessewaites
Copy link
Author

Can you reflect that in the readme and tell us specifically how to use your
version in our gemfiles? I think the readme is a duplicate of the original.
thank you for your hard work!!

On Mon, Sep 8, 2014 at 4:37 PM, Caio Almeida [email protected]
wrote:

FYI, I have a fork with a Rails 4 branch... all tests are passing and I'm
using it on a Rails4 + Ruby 2 application without problems... you can try
that and let me know if things are not working there:
https://github.com/caiosba/activerecord-reputation-system/tree/rails4


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

Jesse Waites
LinkedIn http://www.linkedin.com/profile/view?id=69029327

@caiosba
Copy link
Contributor

caiosba commented Sep 8, 2014

Oh, yes, sure... just did that :)

@jessewaites
Copy link
Author

Awesome - Thanks!!

On Mon, Sep 8, 2014 at 4:51 PM, Caio Almeida [email protected]
wrote:

Oh, yes, sure... just did that :)


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

Jesse Waites
LinkedIn http://www.linkedin.com/profile/view?id=69029327

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants