Skip to content

Rails 4 fix #62

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Rails 4 fix #62

wants to merge 1 commit into from

Conversation

DerKobe
Copy link

@DerKobe DerKobe commented Apr 14, 2015

To work in Rails 4.x views (experiences the Problem with Rails 4.2 / not sure if it's also a problem for 4.0 and 4.1).

@hgmnz
Copy link
Owner

hgmnz commented Apr 14, 2015

I haven't used rails in a while, so looking for a few +1s from the community before merging this!

/cc @zapnap @parndt and everyone? :)

@parndt
Copy link
Collaborator

parndt commented Apr 14, 2015

It seems fine in theory, though I wonder if we should make this plugin less aggressive by making people put this in their controllers manually? Or would that be too much?

@DerKobe
Copy link
Author

DerKobe commented Apr 15, 2015

That would be fine, too, I guess. Then just put the code I added in the readme as a hint?

@zapnap
Copy link
Collaborator

zapnap commented Apr 15, 2015

I'd vote for that as well. Seems cleaner / less invasive that way, and easy enough.

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.

4 participants