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

Add rails5.1 #148

Merged
merged 2 commits into from May 16, 2017
Merged

Add rails5.1 #148

merged 2 commits into from May 16, 2017

Conversation

syguer
Copy link
Collaborator

@syguer syguer commented Apr 28, 2017

Rails 5.1 has been released 🎉

@syguer syguer requested review from kbrock and zilkey April 28, 2017 13:30
@@ -35,7 +35,7 @@ def cache_key
if Object.const_defined?(:ActiveModel)
model_name.cache_key
else
ActiveSupport::Inflector.tableize(self).downcase
ActiveSupport::Inflector.tableize(self.name).downcase
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know why, but this code has been failed.

@syguer
Copy link
Collaborator Author

syguer commented Apr 30, 2017

@zilkey @kbrock can you check?

@kbrock
Copy link
Collaborator

kbrock commented May 1, 2017

@syguer I currently have no projects that use active_hash.
I believe in the project and want to help where I can, but I have no corpus where I can run this.

Is there something in particular that I can do, or are you asking me to just kick the tires?
Keenan

@syguer
Copy link
Collaborator Author

syguer commented May 2, 2017

@kbrock Thanks for reply. just kick the tires please!

@grosser
Copy link
Contributor

grosser commented May 2, 2017

need to get rid of quoted_id too see rails/rails#27962 (comment)
basically make the method not be defined if it's rails 5.1+

@syguer
Copy link
Collaborator Author

syguer commented May 2, 2017

@grosser Thanks for information. It seems to be just deprecated. Could you create an issue about your problem?

@grosser
Copy link
Contributor

grosser commented May 2, 2017

here ya go: #150
should still be taken care of, deprecations are spammy :D

@syguer
Copy link
Collaborator Author

syguer commented May 3, 2017

@grosser thanks! :)

@syguer syguer merged commit 7576771 into active-hash:master May 16, 2017
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.

None yet

3 participants