-
Notifications
You must be signed in to change notification settings - Fork 210
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Sort not working on master #42
Comments
This seems to have been created in the last commit. Undoing it though throws an ugly exception about InheritedResources::Base not being known. I'll keep digging and if I find a solution, I'll submit a pull request. |
Thanks, a failing spec would also be very appreciated! :) |
I will look to see if we can reuse ActiveAdmin's. Sent from my iPhone On May 29, 2013, at 7:52 AM, Elia Schito notifications@github.com wrote:
|
The issue is in ActiveAdmin::ResourceController::DataAccess#apply_sorting. I tried to monkey patch it but I keep getting the following error which I don't know how to solve.
|
solution works great, thanks @bartoszkopinski |
So I managed to have sorting + filtering working on my branch now but didn't have time to test it. Feel free to use the changes if needed: bartoszkopinski/activeadmin-mongoid@elia:master...master |
@bartoszkopinski can you send me a PR? I'll merge straight away if travis 🚥 agrees :) |
- no-collection case - filtering for sorting - conflict with simple_form
Closing, any discussion should move to #51 |
Although the order is passed in, the table in the index page is not ordered.
The text was updated successfully, but these errors were encountered: