-
-
Notifications
You must be signed in to change notification settings - Fork 804
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
Make ransack work with Rails 7.2 #1503
Conversation
This one closes #1489 too. |
This PR added a breaking change, if you use Rails 7.1 you will get this error, when you were supporting rails 7.2. |
@night91 can you show me the error you are getting? The test suite is passing with 7.1.0, is that the version you are on? |
Also having the problem with Rails 7.1.3.4 & Ransack 4.2.0. Rolling back Ransack hasn't managed to help us though. The Error
Params
Models
|
rails 7.1.3.4 Same error as above
|
My version comparison is incorrect. I will fix it a little later today and cut a new release. Thanks for reporting! |
…urselves. Closes #1491.