Skip to content

Kaminari#116

Merged
gregbell merged 6 commits into
activeadmin:masterfrom
damrbaby:kaminari
Jun 1, 2011
Merged

Kaminari#116
gregbell merged 6 commits into
activeadmin:masterfrom
damrbaby:kaminari

Conversation

@damrbaby

Copy link
Copy Markdown
Contributor

I added support for pagination with Kaminari if a rails project is using it. The advantage of kaminari is that it works with mongoid and other ORMs. If Kaminari is not being used, the default will_paginate implementation will still work.

P.S. I'm going to be taking a stab at getting activeadmin playing nicely with mongoid (and possibly other ORMs via orm_adapter). I think the most difficult task will be getting a meta_search implementation that will work. Any ideas?

@gregbell

Copy link
Copy Markdown
Contributor

Hey thanks. I actually think that we should pick one. I don't have any issue with us switching to Kaminari if the community's behind it. I do like how Kaminari works better than will_paginate.

Thoughts?

@kyriacos

Copy link
Copy Markdown

Good idea,

It's the first thing i checked in the code my self. I tried kaminari on a couple of apps i upgraded and it's very nice to work with.

@plentz

plentz commented May 27, 2011

Copy link
Copy Markdown

+1 for kaminari. it looks a lot more active lately. apart from the arguments already listed, will_paginate don't work with rails 3.1 till now(and since the last commit was at october, 2010, I think they don't have any plans on it).

@tmilewski

Copy link
Copy Markdown

+1 Kaminari

@lucaong

lucaong commented May 28, 2011

Copy link
Copy Markdown

+1 Kaminari
it looks like it is becoming the dominant choice

@mankind

mankind commented May 28, 2011

Copy link
Copy Markdown

+1 kaminari

For wider adoption of active_admin, it is best to go with tools that already support other orms like kaminari.

@knoopx

knoopx commented May 29, 2011

Copy link
Copy Markdown
Contributor

+1 for kaminari replacing will_paginate

@jarinudom

Copy link
Copy Markdown

+1 I like Kaminari too

@gregbell

Copy link
Copy Markdown
Contributor

So I think we should just remove will_paginate and replace it with Kaminari. @mwindwer do you have time to modify the pull request for this? Otherwise I can tackle over the next couple of days.

@damrbaby

Copy link
Copy Markdown
Contributor Author

Just removed will_paginate completely. All cucumber tests are passing.

@gregbell

Copy link
Copy Markdown
Contributor

Sweet! Thanks for rocking on this @mwindwer. I'll get a chance to merge this to master tomorrow morning.

@gregbell gregbell merged commit a5c11f3 into activeadmin:master Jun 1, 2011
@amatsuda

amatsuda commented Jun 1, 2011

Copy link
Copy Markdown

w00t!!
SUPER excited to see this happening as the author of Kaminari ⚡ ⚡
Tell me please if I can do any support for this migration.
Thanks!

@gregbell

gregbell commented Jun 1, 2011

Copy link
Copy Markdown
Contributor

@amatsuda hey thanks for a great project! We'll definitely keep in touch if there's anything we need. So far things have been pretty seamless.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants