Skip to content
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

Override for table name fix for Rails 7.1 #25

Merged
merged 1 commit into from
Sep 13, 2024

Conversation

stanleylhs
Copy link
Member

@stanleylhs stanleylhs commented Sep 12, 2024

Cherrypicked commit from Ransack 4.1.1 for Rails 7.1 as table_name is deprecated.
activerecord-hackery/ransack@eac3c37

Cherrypicked commit from Ransack 4.1.1 for Rails 7.1
activerecord-hackery/ransack@eac3c37
@stanleylhs stanleylhs changed the title Override for table name fix Override for table name fix for Rails 7.1 Sep 12, 2024
@CloCkWeRX
Copy link
Member

Anything stopping us from just updating to 4.1.1+?

@stanleylhs
Copy link
Member Author

Anything stopping us from just updating to 4.1.1+?

This is the issue @steveyken is having
activerecord-hackery/ransack#1403

@steveyken
Copy link
Member

I've created #26 to handle the upgrade from v3 to v4. In the meantime, this is a suitable monkey patch fix to provide Rails 7.1 compatibility.

@steveyken steveyken merged commit 2d57d65 into fatfreecrm:master Sep 13, 2024
@stanleylhs
Copy link
Member Author

Ah, I should have made comments in the code about this being a monkey patch, say removed this after v4 upgrade.

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.

3 participants